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 Fix consul token revocation with namespace specific policies into release/1.14.x #23778

Conversation

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

Backport

This PR is auto-generated from #23010 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.


This PR fixes a problem with the Consul Secrets engine where tokens generated in specific Consul namespaces were unable to be revoked if the token provided to Vault to be used for management had namespace specific policies.

Closes #22895


Overview of commits

@hc-github-team-secure-vault-core hc-github-team-secure-vault-core force-pushed the backport/davidadeleon/fix-consul-token-revoke-with-namespace/simply-resolved-trout branch from 05c239b to c559f8c Compare October 20, 2023 18:36
@github-actions github-actions bot added the hashicorp-contributed-pr If the PR is HashiCorp (i.e. not-community) contributed label Oct 20, 2023
@robmonte robmonte added this to the 1.14.5 milestone Oct 20, 2023
@robmonte robmonte enabled auto-merge (squash) October 20, 2023 18:38
@github-actions
Copy link

Build Results:
All builds succeeded! ✅

@robmonte robmonte merged commit aebe972 into release/1.14.x Oct 20, 2023
@robmonte robmonte deleted the backport/davidadeleon/fix-consul-token-revoke-with-namespace/simply-resolved-trout branch October 20, 2023 18:57
@github-actions
Copy link

CI Results:
All Go tests succeeded! ✅

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.

3 participants