-
Notifications
You must be signed in to change notification settings - Fork 27
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
[Auth] Allow users to login and logout #22
Comments
I suggest that we start with token based auth. I want to work on his ticket. |
@maunga-et thanks. If I remember correctly, work on this has been started and the auth currently uses HTTP auth. Do you mind explaining why you're suggesting we go with token auth and how you would implement it? |
@terrameijar He is mentioning token-based authentication because HTTP auth will authenticate DRF's browsable API but will not authenticate the API endpoints for a different origin. It is not possible to authenticate the API endpoints from a different client other than the browsable API via HTTP Auth. If @maunga-et however, wants to implement token authentication, he should implement 2 endpoints that authenticate a mobile client via Authorization headers and another for web clients(React) via HTTP only cookies. |
@terrameijar it makes it easy to integrate with our clients(web and mobile) and also serves to protect private endpoints. We can use simplejwt for that. |
@maunga-et , I missed your comment above, let's go with token auth, please work on this
@IamTyrone makes sense, thank you. |
Users should have the ability to log in and out of the website. The backend has this functionality ready and the API endpoints for auth are
api-auth/login/
andapi-auth/logout
. The backend currently supports basic HTTP username and password authentication.@IamTyrone let's start with basic HTTP authentication to get the ball rolling and move to something else like server-side tokens or JWT later. What do you think?
CC @mpumzee , do you have any thoughts on auth?
The text was updated successfully, but these errors were encountered: