search

Styria-Digital / django-rest-framework-jwt

JSON Web Token Authentication support for Django REST Framework
https://styria-digital.github.io/django-rest-framework-jwt/
MIT License
191 stars 57 forks source link

get_token_from_request: Catch UnicodeDecodeError #71

Closed jayvdb closed 3 years ago

jayvdb commented 4 years ago

If the authentication header cant be parsed, fall back to cookies or allow other handlers.

Closes https://github.com/Styria-Digital/django-rest-framework-jwt/issues/70

jayvdb commented 4 years ago

I'm not sure this is the best way to catch this; looking forward to others opinions.

fitodic commented 3 years ago

@jayvdb Sorry, I forgot this was still active. The PR looks fine by me and I'm ready to merge it. Could you please add a changelog?

jayvdb commented 3 years ago

ping @fitodic

fitodic commented 3 years ago

@jayvdb 1.17.2 has just been released. Sorry for the delay.