-
-
Notifications
You must be signed in to change notification settings - Fork 9.8k
[Security] Update web-token/jwt-library version and adjust checker parameters #57796
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
[Security] Update web-token/jwt-library version and adjust checker parameters #57796
Conversation
|
Hey! Thanks for your PR. You are targeting branch "7.1" but it seems your PR description refers to branch "7.1 for bug fixes". Cheers! Carsonbot |
|
Maybe we should backport #57694. In tests we don't use version 4 of web-token/jwt-library, but since we don't have a conflict rule for it consumers of symfony/security-http can use it and run into the issue this PR is going to fix. |
OidcTokenHandler
|
Better back port than conflict to me also. |
|
Backport commit done @xabbuh @nicolas-grekas |
OidcTokenHandlerf1e656d to
e8ea16b
Compare
xabbuh
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks like changes from #57767 slipped into this PR
b349c2e to
6009932
Compare
|
Oops, fixed @xabbuh |
6009932 to
536cd72
Compare
|
Thank you @thibaut22200. |
Test failed on my Ubuntu, and on my Windows is OK. Differences between order in construct of Jose/Component/Checker/...