-
Notifications
You must be signed in to change notification settings - Fork 387
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
[gcp] Add logic to handle scalar request.policy values on audit #5231
Conversation
Pinging @elastic/security-external-integrations (Team:Security-External Integrations) |
🌐 Coverage report
|
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.
LGTM, my only suggestion would be adding a pipeline test if we had a sample log to exercise this behavior.
LGTM , Agree to the comment from Taylor to add a pipeline test if possible. |
Package gcp - 2.16.2 containing this change is available at https://epr.elastic.co/search?package=gcp |
…tic#5231) * Add logic to handle scalar request.policy values on audit * Add PR number to changelog * Add test
…tic#5231) * Add logic to handle scalar request.policy values on audit * Add PR number to changelog * Add test
What does this PR do?
Adds logic to move
request.policy
intorequest.policy_value
when it is not an object and avoid indexing errors.Checklist
changelog.yml
file.