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

Backport of Compare public keys on trusted leaf certs, to prevent use of alternate certs with the same/forged serial numbers into release/1.14.x #25654

Conversation

hc-github-team-secure-vault-core
Copy link
Contributor

Backport

This PR is auto-generated from #25649 to be assessed for backporting due to the inclusion of the label backport/1.14.x.

The below text is copied from the body of the original PR.


By comparing public keys, we ensure the caller possessed the corresponding
private key of the role configured cert. The TLS stack will have validated
the signature of the provided certificate.


Overview of commits

@hc-github-team-secure-vault-core hc-github-team-secure-vault-core force-pushed the backport/sgm/vault-24406/certauth-check-nonca-pubkeys/incredibly-classic-impala branch from 2bec1a0 to 48af897 Compare February 26, 2024 22:23
@github-actions github-actions bot added the hashicorp-contributed-pr If the PR is HashiCorp (i.e. not-community) contributed label Feb 26, 2024
@sgmiller sgmiller added this to the 1.14.10 milestone Feb 26, 2024
@sgmiller sgmiller enabled auto-merge (squash) February 26, 2024 22:26
Copy link

Build Results:
All builds succeeded! ✅

Copy link

CI Results:
All Go tests succeeded! ✅

@sgmiller sgmiller merged commit 612a54f into release/1.14.x Feb 26, 2024
75 of 76 checks passed
@sgmiller sgmiller deleted the backport/sgm/vault-24406/certauth-check-nonca-pubkeys/incredibly-classic-impala branch February 26, 2024 22:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hashicorp-contributed-pr If the PR is HashiCorp (i.e. not-community) contributed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants