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.

Sign up for GitHub

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

Jump to bottom

Dependency on old compact_jwt revision #443

Open
bernharddick opened this issue Jul 14, 2024 · 1 comment · May be fixed by #444
Open

Dependency on old compact_jwt revision #443

bernharddick opened this issue Jul 14, 2024 · 1 comment · May be fixed by #444

Comments

@bernharddick
Copy link

bernharddick commented Jul 14, 2024 •
edited
Loading

Is your feature request related to a problem? Please describe.
Hi, while looking into packaging webauthn-rs for Debian I stumbled upon the dependency on an 0.2.x release of compact_jwt. As the change to 0.3.x and upwards includes breaking API changes (i.e. JwsUnverified no longer being available) it cannot be solved by just relaxing the dependency. So I kindly ask to update the project to build with a recent compact_jwt revision.

Describe the solution you'd like
Update dependency to recent version of compact_jwt

Describe alternatives you've considered
Co-packaging an old compact_jwt revision is theoretically possible but should be avoided.
@bernharddick bernharddick changed the title Dependency on old jwt_compact revision Dependency on old compact_jwt revision Jul 14, 2024
@Firstyear
Copy link
Member

First, you should not be packaging crates for a distro. It goes against how Rust works as a language.

I will update compact_jwt however.

This was referenced Jul 17, 2024
Merged
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

20240717 update deps Firstyear/webauthn-rs
2 participants
@Firstyear @bernharddick