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
[api] Use BearerAuthentication #91 #139
[api] Use BearerAuthentication #91 #139
Conversation
Some endpoints require the sending of the user API access | ||
token sent in the form of a "Bearer Token", example: | ||
|
||
.. code-block:: shell | ||
|
||
curl -H "Authorization: Bearer <user-token>" \ | ||
'http://localhost:8000/api/v1/default/account/session/' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed example from below and added it before all User APIs as the example to use Bearer <user-token>
is applicable for multiple User APIs
@@ -126,7 +125,6 @@ def get_serializer(self, *args, **kwargs): | |||
|
|||
|
|||
class AuthorizeView(TokenAuthorizationMixin, APIView): | |||
authentication_classes = (TokenAuthentication,) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed from here because TokenAuthorizationMixin
already contains authenication_classes`
ab83861
to
55f3808
Compare
Pull Request Test Coverage Report for Build 443
💛 - Coveralls |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Closes #91