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

Allow login for users without oauth2 #48

Open
frafra opened this issue Apr 1, 2022 · 0 comments · May be fixed by #52
Open

Allow login for users without oauth2 #48

frafra opened this issue Apr 1, 2022 · 0 comments · May be fixed by #52

Comments

@frafra
Copy link

frafra commented Apr 1, 2022

It would be nice to not redirect the user to OAuth2Controller as soon as they click on the Log in link, but show the usual login page with an extra button for the OAuth2 authentication, or keep two different Log in links, one for OAuth2 users, one for CKAN users. That would allow existing users to log in.

m.connect('/user/login',
controller='ckanext.oauth2.controller:OAuth2Controller',
action='login')

We could have an option just to map such controller to a different URL, so that it does not override the original one. Any opinion on that?

@frafra frafra linked a pull request Apr 13, 2022 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant