Skip to content

roles yml changes for security-analytics plugin #2192

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.

Sign up for GitHub

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

Merged
merged 5 commits into from
Nov 2, 2022
Merged

roles yml changes for security-analytics plugin #2192

Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
fc6fe76
roles yml changes for security-analytics plugin
raj-chak Oct 25, 2022
e83abab
changed to opensearch, added index permission and also in alphabetica…
raj-chak Oct 31, 2022
82e2716
added new line
raj-chak Oct 31, 2022
4bbece6
changed role names to use prefix security_analytics and added one mor…
raj-chak Nov 1, 2022
399269e
added entry for security_analytics_ack_alerts
raj-chak Nov 1, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
35 changes: 35 additions & 0 deletions config/roles.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -255,3 +255,38 @@ point_in_time_full_access:
- '*'
allowed_actions:
- 'manage_point_in_time'

# Allows users to see security analytics detectors and others
security_analytics_read_access:
reserved: true
cluster_permissions:
- 'cluster:admin/opensearch/securityanalytics/alerts/get'
- 'cluster:admin/opensearch/securityanalytics/detector/get'
- 'cluster:admin/opensearch/securityanalytics/detector/search'
- 'cluster:admin/opensearch/securityanalytics/findings/get'
- 'cluster:admin/opensearch/securityanalytics/mapping/get'
- 'cluster:admin/opensearch/securityanalytics/mapping/view/get'
- 'cluster:admin/opensearch/securityanalytics/rule/get'
- 'cluster:admin/opensearch/securityanalytics/rule/search'

Copy link
Member

Choose a reason for hiding this comment

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

Lets also add security_analytics_ack_alerts for the security operators to be able to acknowledge alerts with cluster permission as:

cluster_permissions:
    - 'cluster:admin/opendistro/securityanalytics/alerts/*'

Copy link
Contributor Author

@raj-chak raj-chak Nov 1, 2022 •
edited
Loading

Choose a reason for hiding this comment

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

done as opensearch not opendistro to be consistent with the rest

# Allows users to use all security analytics functionality
security_analytics_full_access:
reserved: true
cluster_permissions:
- 'cluster:admin/opensearch/securityanalytics/alerts/*'
- 'cluster:admin/opensearch/securityanalytics/detector/*'
- 'cluster:admin/opensearch/securityanalytics/findings/*'
- 'cluster:admin/opensearch/securityanalytics/mapping/*'
- 'cluster:admin/opensearch/securityanalytics/rule/*'
index_permissions:
- index_patterns:
- '*'
allowed_actions:
- 'indices:admin/mapping/put'
Copy link
Member

Choose a reason for hiding this comment

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

How about indices:admin/mappings/get ? Also are there more permissions such as for aliases that might be needed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done and tested.

- 'indices:admin/mappings/get'

# Allows users to view and acknowledge alerts
security_analytics_ack_alerts:
reserved: true
cluster_permissions:
- 'cluster:admin/opensearch/securityanalytics/alerts/*'