-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Add FilterLogEventsPolicy to policy templates. #729
Conversation
This commit adds a new policy to the policy templates that grants permission to call the filter-log-events API, given a Log Group.
To be able to access the Log Streams within a Log Group, the * is needed in the filter log events policy, so this commit adds it to the resource field.
…tream:* instead of logGroupName*.
Codecov Report
@@ Coverage Diff @@
## develop #729 +/- ##
========================================
Coverage 94.18% 94.18%
========================================
Files 67 67
Lines 2683 2683
Branches 478 478
========================================
Hits 2527 2527
Misses 80 80
Partials 76 76 Continue to review full report at Codecov.
|
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.
Thanks for doing this! Please also add this policy to the all_policy_templates.yaml example. See #715 as an example.
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.
Thanks for this update!
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.