Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

T5683: Fix reverse-proxy PKI filenames mismatch (backport #2405) #2407

Merged
merged 1 commit into from
Oct 26, 2023

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Oct 25, 2023

This is an automatic backport of pull request #2405 done by Mergify.


Mergify commands and options

More conditions and actions can be found in the documentation.

You can also trigger Mergify actions by commenting on this pull request:

  • @Mergifyio refresh will re-evaluate the rules
  • @Mergifyio rebase will rebase this PR on its base branch
  • @Mergifyio update will merge the base branch into this PR
  • @Mergifyio backport <destination> will backport this PR on <destination> branch

Additionally, on Mergify dashboard you can:

  • look at your merge queues
  • generate the Mergify configuration with the config editor.

Finally, you can contact us on https://mergify.com

The current named for certificates are hardcoded in generated config to:
 - ca.pem
 - cert.pem.key
 - cert.pem

It cause a generated config certificates and certificates itself
are different (test-cert-1.pem and ca.pem)

  bind :::8080 v4v6 ssl crt /run/haproxy/test-cert-1.pem

  /run/haproxy/ca.pem

It is a bug of initial impelemtation. Fix required correct names
from PKI certificates

(cherry picked from commit 0431f1b)
@vyosbot vyosbot requested review from a team, dmbaturin, sarthurdev, zdc, jestabro, sever-sever and c-po and removed request for a team October 25, 2023 18:26
@github-actions github-actions bot added the sagitta VyOS 1.4 LTS label Oct 25, 2023
@sever-sever sever-sever merged commit 6c514d7 into sagitta Oct 26, 2023
4 checks passed
@mergify mergify bot deleted the mergify/bp/sagitta/pr-2405 branch October 26, 2023 02:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sagitta VyOS 1.4 LTS
Development

Successfully merging this pull request may close these issues.

3 participants