Allow non-admin users to log in to the admin UI #122
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Get the user's groups on login and store them with their credentials in the ES cookie Don't start the $scavenges poll if the user is not an admin user Show a message in the stream browser telling non-admin users they can't view recently created streams Don't query for streams/$all if the user is not admin Hide 'Edit ACL' button for non-admin users on streams Check if a stream exists and that the user has access to it before going to a searched-for stream. Disable all tabs except dashboard, streams and queries for non-admin users Disable nav items when they are unavailable instead of hiding them and show a tooltip explaining why it's disabled Return to the correct state (query or projection) when going back from debugging a projection and reload the query if you are going back to the query state from the projections debugging state. Include the state reason for instance when a projection fails because of an authorisation exception, it's useful to know why it failed
pgermishuys
force-pushed
the
non-admin-users
branch
from
e0465d6 to
42e4785
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes kurrent-io/KurrentDB#873
Get the user's groups on login and store them with their credentials in the ES cookie.
Don't start the $scavenges poll if the user is not an admin user.
Show a message in the stream browser telling non-admin users they can't view recently created streams.
Don't query for streams/$all if the user is not an admin.
Hide 'Edit ACL' button for non-admin users on streams.
Check if a stream exists and that the user has access to it before going to a searched-for stream.
Disable all tabs except dashboard, streams and queries for non-admin users.
Disable nav items when they are unavailable instead of hiding them and show a tooltip explaining why it's disabled.
Return to the correct state (query or projection) when going back from debugging a projection
and reload the query if you are going back to the query state from the projections debugging state.