Skip to content

Implement access monitoring review handler#53773

Merged
bernardjkim merged 5 commits into
masterfrom
bernard/accessmonitoring-review
Apr 12, 2025
Merged

Implement access monitoring review handler#53773
bernardjkim merged 5 commits into
masterfrom
bernard/accessmonitoring-review

Conversation

@bernardjkim
Copy link
Copy Markdown
Contributor

Supports: #51682
RFD: #51979

This PR implements the access review handler. The access review handler stores a cache of access monitoring rules, and is responsible for submitting access reviews when a matching access request is observed.

These changes are split from a larger PR. To see how it'll be used with the access monitor, see #52988.

@bernardjkim bernardjkim added the no-changelog Indicates that a PR does not require a changelog entry label Apr 7, 2025
Comment thread lib/accessmonitoring/review/review.go
Comment thread lib/accessmonitoring/review/review.go
Comment thread lib/accessmonitoring/cache.go Outdated
Comment thread lib/accessmonitoring/review/review.go Outdated
@bernardjkim bernardjkim enabled auto-merge April 12, 2025 04:14
@bernardjkim bernardjkim added this pull request to the merge queue Apr 12, 2025
Merged via the queue into master with commit c32fa77 Apr 12, 2025
39 checks passed
@bernardjkim bernardjkim deleted the bernard/accessmonitoring-review branch April 12, 2025 04:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automatic-reviews no-changelog Indicates that a PR does not require a changelog entry size/lg

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants