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

Dependency jwk-rs is unmaintained #112

Closed
howardjohn opened this issue Sep 19, 2024 · 3 comments · Fixed by #115
Closed

Dependency jwk-rs is unmaintained #112

howardjohn opened this issue Sep 19, 2024 · 3 comments · Fixed by #115

Comments

@howardjohn
Copy link
Contributor

Currently https://github.com/nhynes/jwk-rs is a dependency, pulling in some critical crypto operations (ring, etc), but is seemingly abandoned. This transitively pulls in things like a 3 year old version of ring, etc, which is not great. Maintenance PRs against the dep are going unmerged for years: nhynes/jwk-rs#13, etc.

It would be good to plan an alternative strategy for this dependency

@maxlambrecht
Copy link
Owner

hi @howardjohn, thanks for pointing this out. I see the issue with relying on an unmaintained library, especially for critical crypto operations. I’ll look into other options.

Do you have any alternative libraries in mind that could replace jwk-rs?

@howardjohn
Copy link
Contributor Author

Do you have any alternative libraries in mind that could replace jwk-rs?

At the moment I don't, I don't have a ton of experience in this space. There may be one, haven't had a chance to explore much.

@bleggett
Copy link
Collaborator

bleggett commented Sep 30, 2024

We should maybe just use https://github.com/Keats/jsonwebtoken directly.

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

Successfully merging a pull request may close this issue.

3 participants