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

Audit: logging a response uses a separate 5 second timeout #24238

Merged
merged 3 commits into from
Nov 22, 2023

Conversation

peteski22
Copy link

@peteski22 peteski22 commented Nov 22, 2023

This PR addresses a potential issue with not attempting to audit log a response in which the context is already Done.

Should resolve: #23871

(5 seconds was a sensible value that came from a conversation with @ncabatoff)

@peteski22 peteski22 added core/audit hashicorp-contributed-pr If the PR is HashiCorp (i.e. not-community) contributed labels Nov 22, 2023
@peteski22 peteski22 added this to the 1.15.3 milestone Nov 22, 2023
Copy link

github-actions bot commented Nov 22, 2023

CI Results:
All Go tests succeeded! ✅

@peteski22 peteski22 requested review from ncabatoff and a team November 22, 2023 18:07
@peteski22 peteski22 added the bug Used to indicate a potential bug label Nov 22, 2023
@peteski22 peteski22 marked this pull request as ready for review November 22, 2023 18:07
Copy link

Build Results:
All builds succeeded! ✅

Copy link
Contributor

@raskchanky raskchanky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@@ -10,6 +10,8 @@ import (
"sync"
"time"

"github.com/hashicorp/vault/helper/namespace"

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The extra blank line strikes again!

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

But the cat came back the very next day,
the cat came back, they thought he was a goner.
But the cat came back; it just couldn’t stay away.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#24248 fixes #24238 😜

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Used to indicate a potential bug core/audit 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.

Error "event not processed by enough 'sink' nodes" when file audit enabled in Vault 1.15
2 participants