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 VAULT-14734: activity log write endpoint into release/1.11.x #20134

Conversation

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

Backport

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

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


This PR creates a no-op endpoint at sys/internal/counters/activity/write for writing activity log data when Vault is compiled with the testonly build tag. This endpoint accepts JSON input for the data to write to the activity log.

The input is defined as protobuf and Go models are generated from the protobuf for simplicity's sake - there are several "oneof" options in the input, and protobuf gives a built-in way to handle those restrictions.

Currently, the endpoint only validates the input. A later PR will update the handler to generate and write the data.


Overview of commits

@hc-github-team-secure-vault-core hc-github-team-secure-vault-core force-pushed the backport/miagilepner/VAULT-14734-activity-log-write-endpoint/nationally-deciding-dog branch 2 times, most recently from 6c8b357 to 589b20d Compare April 13, 2023 10:32
@miagilepner miagilepner force-pushed the backport/miagilepner/VAULT-14734-activity-log-write-endpoint/nationally-deciding-dog branch from 589b20d to c6e92dd Compare April 14, 2023 07:36
@miagilepner miagilepner enabled auto-merge (squash) April 14, 2023 07:37
@miagilepner miagilepner merged commit 4397b15 into release/1.11.x Apr 14, 2023
@miagilepner miagilepner deleted the backport/miagilepner/VAULT-14734-activity-log-write-endpoint/nationally-deciding-dog branch April 25, 2023 09:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants