-
Notifications
You must be signed in to change notification settings - Fork 24.3k
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
Security - kibana_system - add delete privileges for synthetics #85844
Changes from 4 commits
3541baa
8a2173d
7dfe447
b9e6578
b27cc86
6580ad0
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,5 @@ | ||
pr: 85844 | ||
summary: Add delete privilege to `kibana_system` for Synthetics | ||
area: Authorization | ||
type: enhancement | ||
issues: [] |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -777,7 +777,13 @@ public static RoleDescriptor kibanaSystemRoleDescriptor(String name) { | |
"metrics-apm-*", | ||
"metrics-apm.*-*", | ||
"traces-apm-*", | ||
"traces-apm.*-*" | ||
"traces-apm.*-*", | ||
"synthetics-http-*", | ||
"synthetics-icmp-*", | ||
"synthetics-tcp-*", | ||
"synthetics-browser-*", | ||
"synthetics-browser.network-*", | ||
"synthetics-browser.screenshot-*" | ||
Comment on lines
+785
to
+790
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. question: do we document these patterns anywhere like we do for APM patterns ( There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Unfortunately no, we do have this docs ticket where we will address in the future elastic/synthetics#286 |
||
) | ||
.privileges(DeleteIndexAction.NAME) | ||
.build(), | ||
|
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.
nit: it'd make sense to mention
Synthetics
in the comment above.