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

PyJWT==2.2.0 #467

Closed
garyburgmann opened this issue Oct 7, 2021 · 6 comments · Fixed by #472
Closed

PyJWT==2.2.0 #467

garyburgmann opened this issue Oct 7, 2021 · 6 comments · Fixed by #472

Comments

@garyburgmann
Copy link

new release today:

image

causing issues as referenced by:

#464 (comment)

pinning PyJWT<2.2 or alternatively PyJWT==2.1.0 in personal projects is a workaround for now, but ideally this would be either pinned appropriately here:

https://github.com/jazzband/djangorestframework-simplejwt/blob/master/setup.py#L59

or handled in source code

@lsaavedr
Copy link

lsaavedr commented Oct 7, 2021

configmed!

@garyburgmann
Copy link
Author

@Andrew-Chen-Wang re:

#464 (comment)

I can't do this in the short term but if someone doesn't pick it up by the weekend I will handle this

@Artem19861910
Copy link
Contributor

Confim, same with both 4.7.2 (we used so far) and 4.8.0. In requirements we've hardcoded PyJWT==2.1.0 and "miracle" happened.

@anushbmx
Copy link

anushbmx commented Oct 7, 2021

PyJWT 2.2.0 is causing the following error "decode() got an unexpected keyword argument 'verify'" in here

Pinning PyJWT==2.1.0 is the quick fix.

@easy-ms
Copy link

easy-ms commented Oct 7, 2021

Confirmed.

return jwt.decode(token, self.verifying_key, algorithms=[self.algorithm], verify=verify,
TypeError: decode() got an unexpected keyword argument 'verify'

Quick fix is to set PyJWT==2.1.0 in your requirements.

jerr0328 added a commit to jerr0328/djangorestframework-simplejwt that referenced this issue Oct 7, 2021
Fixes jazzband#467 temporarily until the code change can be made and validated
sainak added a commit to flaam-org/flaam-api that referenced this issue Oct 7, 2021
@Andrew-Chen-Wang Andrew-Chen-Wang pinned this issue Oct 7, 2021
@pikuni
Copy link

pikuni commented Oct 8, 2021

Fixed the issue with PyJWT==2.1.0 and djangorestframework-simplejwt==4.8.0 in requirements.txt

PyJWT==2.1.0
djangorestframework-simplejwt==4.8.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
6 participants