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

How to proceed if RBAC prohibits viewing of a key? #96

Closed
tuxmea opened this issue Oct 18, 2022 · 4 comments · Fixed by #111
Closed

How to proceed if RBAC prohibits viewing of a key? #96

tuxmea opened this issue Oct 18, 2022 · 4 comments · Fixed by #111
Assignees

Comments

@tuxmea
Copy link
Member

tuxmea commented Oct 18, 2022

Should HDM show that access to the key is denied?
Should HDM show nothing?

@tuxmea tuxmea self-assigned this Oct 18, 2022
@TwizzyDizzy
Copy link

TwizzyDizzy commented Oct 30, 2022

From a UX perspective it makes sense to:

  • let the user know that there is something rather than simply omitting that information silently and thereby sowing confusion
  • visually indicate, that they key is there, but cannot be accessed. Suggestion: grey out that key or indicate permission issues by even highlighting it in a signal color like e.g. red

Cheers
Thomas

@rwaffen
Copy link
Member

rwaffen commented Oct 31, 2022

we talked about this already and looked at it from a security perspective: when you are not allowed to see something, you should not know that there is something 🤔 like it is in other products.

@tuxmea
Copy link
Member Author

tuxmea commented Oct 31, 2022

@TwizzyDizzy many thanks for the comment.
As @rwaffen already mentioned: we had an internal discussion on this topic.

Arguments against showing presence of existing data without access:

  • knowing that there is a secret is a security violation
  • letting people know that there is a secret which they can not access causes irritation and potentially let people hack around, getting the data.

Arguments for showing that there is a secret without access:

  • easy to request access to

We had a look at other tools and we learned that most tools will not show any occurrence of non-accessible data.

Please let us know if you see other reasons on why to show hidden data.

@tuxmea
Copy link
Member Author

tuxmea commented Nov 8, 2022

No further comment.
@oneiros Solution: show hint: You are either not allowed to access the data or the requested data do not exist.

@tuxmea tuxmea assigned oneiros and unassigned tuxmea Nov 8, 2022
@oneiros oneiros mentioned this issue Dec 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants