Skip to content
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

[Monitoring] Only elevated permission users should see Setup Mode #50327

Closed
chrisronline opened this issue Nov 12, 2019 · 3 comments · Fixed by #50421
Closed

[Monitoring] Only elevated permission users should see Setup Mode #50327

chrisronline opened this issue Nov 12, 2019 · 3 comments · Fixed by #50421
Labels
bug Fixes for quality problems that affect the customer experience Feature:Stack Monitoring Team:Monitoring Stack Monitoring team

Comments

@chrisronline
Copy link
Contributor

Right now, all monitoring users are able to enter Setup Mode, but this should be restricted.

Currently, the basic monitoring user will run into errors with the following permissions:

We need to figure out a way to detect these privileges for the current user and enable/disable the setup mode UI accordingly.

@chrisronline chrisronline added bug Fixes for quality problems that affect the customer experience Team:Monitoring Stack Monitoring team Feature:Stack Monitoring labels Nov 12, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/stack-monitoring (Team:Monitoring)

@chrisronline
Copy link
Contributor Author

Screen Shot 2019-11-12 at 11 03 24 AM

This permission set would work for Setup Mode

@chrisronline
Copy link
Contributor Author

chrisronline commented Nov 12, 2019

Another note here.

@jasontedor recommended migrating away from _cat/indices (all _cat apis in general too) and try something like _all as a way to get the list of indices, as used by this call

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience Feature:Stack Monitoring Team:Monitoring Stack Monitoring team
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants