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.

Sign up for GitHub

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

Jump to bottom

Enable ckan auth #52

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Enable ckan auth #52

wants to merge 2 commits into from

Conversation

frafra
Copy link

@frafra frafra commented Apr 13, 2022 •
edited

Fix #48.

@frafra frafra marked this pull request as ready for review July 14, 2022 16:42
@frafra
Copy link
Author

frafra commented Jul 17, 2022

Any comment on this @aarranz?

@frafra
Copy link
Author

frafra commented Jul 17, 2022

It would also be useful to be able to change users' settings, since now it is not possible to make someone admin, for example, or change password for non-oauth2 users.

@frafra
Copy link
Author

frafra commented Aug 19, 2022

@aitormagan any opinion on that?

@aitormagan
Copy link
Contributor

aitormagan commented Aug 22, 2022 •
edited

The main intention when this plugin was developer was to avoid the CKAN native auth. But this can also be a good idea. However, I would say it would be interesting to let developers to choose the mode they want their instance to work in order not to break backwards compatibility.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Allow login for users without oauth2
2 participants
@frafra @aitormagan