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

Feature request: Implement oauth2 provider for third party access #112

Open
alopix opened this issue Aug 27, 2017 · 3 comments
Open

Feature request: Implement oauth2 provider for third party access #112

alopix opened this issue Aug 27, 2017 · 3 comments

Comments

@alopix
Copy link

alopix commented Aug 27, 2017

I'm a little worried about the latest additions to the API – especially the possibility to login using the username and password, which I think is a huge risk. Sharing your password with a 3rd party app, it will require a lot of trust for the 3rd party.
I would suggest switching to OAuth2 instead so the user is secure and does not have to share the password with 3rd party API users

@ubergesundheit
Copy link
Member

Hi @alopix,

could you please enlighten us which use case you have in mind specifically for third party access?

Currently username and password is only used to obtain a JSON web token which is then used to authenticate user routes. Server side, your password is stored as a bcrypt hash. There is no need to share your credentials with someone else.

@alopix
Copy link
Author

alopix commented Aug 28, 2017

When using the API as a 3rd party, I can either ask the user to enter the API auch key they received during the sign up process (if those old keys still work) or let him login via the provided API call, where the user basically has to share the username & password with the 3rd party (=me) as they will not call the POST request themselves. Otherwise creating an access token is not possible.

This is how the plain text password will be shared with 3rd parties, who can then just do whatever they want (if they were malicious).

@ubergesundheit ubergesundheit changed the title Use OAuth2 instead of sharing the password Feature request: Implement oauth2 provider for third party access Aug 28, 2017
@ubergesundheit
Copy link
Member

The old api-keys are not used anymore.

Implementing an oauth2 provider is currently not on the roadmap, but I would be happy to review a pull request implementing such addition.

I'll change the title of this issue to "Feature request: Implement oauth2 provider for third party access"

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

2 participants