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

Unmaintained dependency python-jose #7244

Closed
estyrke opened this issue Jan 23, 2024 · 6 comments · Fixed by #7356
Closed

Unmaintained dependency python-jose #7244

estyrke opened this issue Jan 23, 2024 · 6 comments · Fixed by #7356

Comments

@estyrke
Copy link

estyrke commented Jan 23, 2024

Moto with extra cognitoidp depends on python-jose, which in turn depends on ecdsa, which in turn was flagged by dependabot: GHSA-wj6h-64fc-37mp

It doesn't seem like there will be a fix in python-ecdsa and python-jose hasn't had any releases for years, so I am wondering if it is possible to replace python-jose with joserfc or something similar? I could try to make a PR if it sounds reasonable. :)

@bblommers
Copy link
Collaborator

Hi @estyrke, that makes sense - looks like joserfc is indeed a much more active library.

PR's are always welcome! Please base any work on this on the v5 branch, as this change is unlikely to make it into a v4 release.

Let me know if you need any help with that.

@bblommers
Copy link
Collaborator

Please base any work on this on the v5 branch, as this change is unlikely to make it into a v4 release.

Actually - this branch has already been merged into master, so you can just base it off of that as normal.

@seanmerrifield
Copy link

Also just noticed this issue and it's failing our vulnerability scans. Would be great if we can remove the ecdsa dependency

@estyrke
Copy link
Author

estyrke commented Jan 30, 2024

Turns out that ecdsa is pulled in through more dependencies than just python-jose. I have a PR in the works, but it got a bit more complex than expected...

@estyrke
Copy link
Author

estyrke commented Feb 18, 2024

Thanks for picking up the slack @bblommers ! I haven't had much spare time lately. 😅

@bblommers
Copy link
Collaborator

No problem @estyrke! The ecdsa dependency has been removed in Moto 5.0.2, just released.

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