-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
VAULT-25710: Audit - enforce header formatter requirement in EntryFormatter #26239
VAULT-25710: Audit - enforce header formatter requirement in EntryFormatter #26239
Conversation
…er in audit entry formatter node
CI Results: |
Build Results: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Found a serious thing in vault/audited_headers.go that needs to be addressed. I'm using Comment rather than Request changes so that once it's fixed someone in your timezone can go over things and provide an Approve review.
…/unseal-and-headerformatter
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR makes changes to the
EntryFormatter
node so that it requires aHeaderFormatter
rather than accepting it as an option.ENT PR: https://github.com/hashicorp/vault-enterprise/pull/5680