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

federator: Expect a client certificate to be the certificate chain #4089

Merged
merged 2 commits into from
Jun 13, 2024

Conversation

akshaymankar
Copy link
Member

Without this openssl doesn't forward to whole chain causing mTLS to not succeed.

https://wearezeta.atlassian.net/browse/WPB-9692

Checklist

  • Add a new entry in an appropriate subdirectory of changelog.d
  • Read and follow the PR guidelines

Without this openssl doesn't forward to whole chain causing mTLS to not succeed.
@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Jun 12, 2024
@akshaymankar akshaymankar merged commit 30abe72 into develop Jun 13, 2024
8 of 9 checks passed
@akshaymankar akshaymankar deleted the federator-client-cert-chain branch June 13, 2024 06:59
akshaymankar added a commit that referenced this pull request Jun 13, 2024
…4089)

Without this openssl doesn't forward to whole chain causing mTLS to not succeed.
akshaymankar added a commit that referenced this pull request Jun 17, 2024
…4092)

Without this openssl doesn't forward to whole chain causing mTLS to not succeed.

Backport of #4089
@echoes-hq echoes-hq bot added echoes: product-roadmap Work aligned with the customer-announced roadmap, targeting a specific release date. echoes: unplanned Any work item that isn’t part of the product or technical roadmap. labels Jun 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
echoes: product-roadmap Work aligned with the customer-announced roadmap, targeting a specific release date. echoes: unplanned Any work item that isn’t part of the product or technical roadmap. ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants