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

[dev.icinga.com #13473] Restrictions #11

Closed
icinga-migration opened this issue Dec 8, 2016 · 1 comment · Fixed by #52
Closed

[dev.icinga.com #13473] Restrictions #11

icinga-migration opened this issue Dec 8, 2016 · 1 comment · Fixed by #52
Assignees
Labels
enhancement New feature or improvement

Comments

@icinga-migration
Copy link

This issue has been migrated from Redmine: https://dev.icinga.com/issues/13473

Created by dgoetz on 2016-12-08 10:36:43 +00:00

Assignee: (none)
Status: New
Target Version: (none)
Last Update: 2016-12-08 10:36:43 +00:00 (in Redmine)


I would like to have restrictions applied to the Graphite Overview so a user can only select hosts he could access in the monitoring and other modules, too.

@icinga-migration icinga-migration added the enhancement New feature or improvement label Jan 17, 2017
@Al2Klimov Al2Klimov self-assigned this Sep 20, 2017
Al2Klimov added a commit that referenced this issue Sep 20, 2017
Al2Klimov added a commit that referenced this issue Sep 20, 2017
@Al2Klimov
Copy link
Member

Note for the future myself: The only needed, but not secured action is /show/graph which gets already transformed monitored objects names.

Solution:

  1. reverse transform to pattern
  2. query IDO by this pattern
  3. check the (already restricted) result and restrict respectively

If one really has the hosts mysql1.mybase and mysql1_mybase – it's their problem.

@Al2Klimov Al2Klimov added the wip label Sep 21, 2017
Al2Klimov added a commit that referenced this issue Sep 22, 2017
Al2Klimov added a commit that referenced this issue Sep 22, 2017
Al2Klimov added a commit that referenced this issue Sep 22, 2017
Al2Klimov added a commit that referenced this issue Sep 22, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or improvement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants