-
Notifications
You must be signed in to change notification settings - Fork 39
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
Switch from python-jose to pyjwt #42
Comments
@siebediels i just tested locally these modifications in #41 and they just work out-of-the-box. there could be some improvements regarding i'd just merge that one PR and go ahead |
if needed - i can make a PR. for now i have a working local version of code. just ask ;) |
aaight... lezz do this #43 |
i've made some updates and some more fixes to the PR. enforced some verifications by default, unless a developer overrides them explicitly. additionally, i'd like to remove the |
bump. unfortunately, no activity on my PR :( |
Is there a plan to migrate from
python-jose
topyjwt
? Python-jose isn't maintained any more and contains some known vulnerabilities.I noticed that there was some effort done in #41 , but not sure what happened to it. As an intermediate solution, we could perhaps move to
python-jose[cryptography]
which is already recommended above the defaultpython-jose
(with Python backend)?The text was updated successfully, but these errors were encountered: