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

VAULT-14734: activity log write endpoint #20019

Merged
merged 6 commits into from
Apr 12, 2023

Conversation

miagilepner
Copy link
Contributor

@miagilepner miagilepner commented Apr 6, 2023

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.

@miagilepner miagilepner added this to the 1.14 milestone Apr 6, 2023
@miagilepner miagilepner requested review from ncabatoff, mpalmi and a team April 6, 2023 14:34
@miagilepner miagilepner marked this pull request as ready for review April 6, 2023 15:31
@miagilepner miagilepner force-pushed the miagilepner/VAULT-14734-activity-log-write-endpoint branch from b002e32 to 5ef2146 Compare April 11, 2023 07:05
Copy link
Contributor

@hghaf099 hghaf099 left a comment

Choose a reason for hiding this comment

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

Looks good! Just a few comments/questions.

Makefile Show resolved Hide resolved
Makefile Outdated Show resolved Hide resolved
vault/activity/generation/generate_data.proto Outdated Show resolved Hide resolved
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.

Looks good! I gave a 👍 to some of Hamid's comments that I agreed with.

Copy link
Contributor

@hghaf099 hghaf099 left a comment

Choose a reason for hiding this comment

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

Looks great!

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.

3 participants