Skip to content
This repository was archived by the owner on Jul 22, 2021. It is now read-only.

[NIFIREG-30] add login component, auth guard to login, tenants, bucke…#58

Closed
scottyaslan wants to merge 1 commit intoapache:masterfrom
scottyaslan:NIFIREG-30
Closed

[NIFIREG-30] add login component, auth guard to login, tenants, bucke…#58
scottyaslan wants to merge 1 commit intoapache:masterfrom
scottyaslan:NIFIREG-30

Conversation

@scottyaslan
Copy link
Contributor

…ts, admin, and explorer routes. Also added route for login component. [NIFIREG-20] improve page load times. [NIFIREG-15] add loading screen.

In an unsecured registry an anonymous user should be able to access:
http://localhost:8080/nifi-registry/login
http://localhost:8080/nifi-registry/administration/workflow
http://localhost:8080/nifi-registry/explorer/grid-list

an attempt to access http://localhost:8080/nifi-registry/administration/users will reroute the user to the login screen (as will any other attempt to access a deep link to which a user does not have access to).

an attempt to access an unknown route (http://localhost:8080/nifi-registry/abcef) will prompt the user with a dialog to click to easily access http://localhost:8080/nifi-registry/explorer/grid-list

In a secured registry the user should only have access to the routes for which they have permissions. If the user is logging in via certs or kerberos the client recognizes this and routes the user to the http://localhost:8080/nifi-registry/explorer/grid-list.

…ts, admin, and explorer routes. Also added route for login component. [NIFIREG-20] improve page load times. [NIFIREG-15] add loading screen.
@bbende
Copy link
Contributor

bbende commented Dec 15, 2017

Functionality looks good, awesome to control the user privileges through the UI in secure mode! Going to merge this

@asfgit asfgit closed this in 580f775 Dec 15, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants