cduvray / jwt-authorizer

JWT authorization layer for Axum.
MIT License
69 stars 21 forks source link

Update jsonwebtoken to latest version #40

Closed blablacio closed 12 months ago

blablacio commented 1 year ago

Updates jsonwebtoken to latest version and also updates key algorithm handling when initializing key data.

Should close #25.

cduvray commented 1 year ago

Hi, thanks for this PR! I think I will try to "regenerate" the test tokens without audiences (and with audiences for some tests), because it breaks too many tests. I will push it on the main branch you can then rebase on it.

blablacio commented 1 year ago

Hey, just noticed I missed one test in tonic. I think it's okay to not regenerate test tokens. The changes are required only because aud is validated against if it's present in the token in newer versions of jsonwebtoken.

cduvray commented 12 months ago

already added the tokens ;), this is also to test more more precisely with and without auditions.

blablacio commented 12 months ago

Seems tests are still passing without any changes on my side :)