Skip to content

Conversation

abhishekbhatia1710
Copy link

Summary

This pull request updates the reserved roles in the security authorization store to include access to a new monitoring index for privileged users. The main changes are focused on expanding the indices that the viewer and editor roles can access.

Role privilege updates:

  • Added a new index pattern constant PRIVILEGED_USER_MONITORING_INDEX to ReservedRolesStore for privileged user monitoring (.entity_analytics.monitoring*).

@abhishekbhatia1710 abhishekbhatia1710 self-assigned this Sep 12, 2025
@abhishekbhatia1710 abhishekbhatia1710 requested a review from a team as a code owner September 12, 2025 09:15
@elasticsearchmachine elasticsearchmachine added needs:triage Requires assignment of a team area label v9.2.0 external-contributor Pull request authored by a developer outside the Elasticsearch team labels Sep 12, 2025
@abhishekbhatia1710 abhishekbhatia1710 added the Team:Security Meta label for security team label Sep 12, 2025
@elasticsearchmachine elasticsearchmachine removed the Team:Security Meta label for security team label Sep 12, 2025
@abhishekbhatia1710 abhishekbhatia1710 added the :Security/Security Security issues without another label label Sep 15, 2025
@elasticsearchmachine elasticsearchmachine added Team:Security Meta label for security team and removed needs:triage Requires assignment of a team area label labels Sep 15, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-security (Team:Security)

Copy link
Contributor

@richard-dennehy richard-dennehy left a comment

Choose a reason for hiding this comment

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

Could you add some tests for the new privilege to ReservedRolesStoreTests?

@@ -849,7 +851,7 @@ private static RoleDescriptor buildEditorRoleDescriptor() {
.build(),
// Security - Entity Store is view only
Copy link
Contributor

Choose a reason for hiding this comment

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

Nit: this comment is out of date, as the following code now grants read-only access to multiple indices

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
external-contributor Pull request authored by a developer outside the Elasticsearch team >non-issue :Security/Security Security issues without another label Team:Security Meta label for security team v9.2.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants