search

iMerica / dj-rest-auth

Authentication for Django Rest Framework
https://dj-rest-auth.readthedocs.io/en/latest/index.html
MIT License
1.63k stars 304 forks source link

Modify JWT Serializer Field Names #501

Closed Dresdn closed 1 year ago

Dresdn commented 1 year ago

Closes #142

This change modifies the JWTSerializer fields to be access and response to align with the naming of rest_framework_jwt.serializers.TokenRefreshSerializer

With rest_framework_simplejwt being tightly coupled into the JWT usage, this made sense to adhere to the upstream dependency naming standards.

Ultimately, it's ugly when using JWT to check for access_token names when authenticating, and then using access for a refresh.

Dresdn commented 1 year ago

Good call @zackkh. I saw that but didn't dawn on me to include it. Let me know if there's anything else to be done.