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

Change login endpoint to use username instead of login. #154

Open
tisto opened this issue Oct 25, 2016 · 4 comments
Open

Change login endpoint to use username instead of login. #154

tisto opened this issue Oct 25, 2016 · 4 comments

Comments

@tisto
Copy link
Sponsor Member

tisto commented Oct 25, 2016

I just ran into this when debugging the login. The "@login" endpoint uses "login" instead of "username". Since Plone itself uses "username", I was confused at first.

@buchi can you elaborate why we use "login"/"password" instead of "username"/"password"?

@buchi
Copy link
Member

buchi commented Nov 7, 2016

@tisto "login" is the term used in PluggableAuthService. Unfortunately the naming is not very consistent in Plone.

I've not objections against renaming it to "username".

@tisto tisto changed the title Login endpoint uses login/password instead of username/password. Change login endpoint to use username instead of login. Nov 26, 2016
@tisto
Copy link
Sponsor Member Author

tisto commented Nov 26, 2016

@buchi ok, then I would like to change that. We can still accept "login" to keep backwards compatibility. At least until we do a 1.0 release.

@jaroel
Copy link
Member

jaroel commented Mar 4, 2017

We were discussion this in plone.api too, but the other way around. Currently that path is dead in the water, and I think we are not going to go down that road.

See plone/plone.api#270 and plone/plone.api#204 (comment) .

@jaroel
Copy link
Member

jaroel commented Mar 4, 2017

btw, does login or username also work for login using email?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants