Skip to content

[SecurityBundle] Update web-token/jwt-library version and adjust checker parameters #57694

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

Merged
merged 1 commit into from
Jul 10, 2024

Conversation

Spomky
Copy link
Contributor

@Spomky Spomky commented Jul 9, 2024

Q A
Branch? 7.2
Bug fix? no
New feature? no
Deprecations? no
Issues Fix nothing
License MIT

The web-token/jwt-library has been updated to allow versions up to 4.0 across multiple components. Additionally, the parameters for the IssuedAtChecker, NotBeforeChecker, and ExpirationTimeChecker in the OidcTokenHandler have been adjusted.

The web-token/jwt-library has been updated to allow versions up to 4.0 across multiple components. Additionally, the parameters for the IssuedAtChecker, NotBeforeChecker, and ExpirationTimeChecker in the OidcTokenHandler have been adjusted to support named arguments as required by the new version of the library.
@fabpot
Copy link
Member

fabpot commented Jul 10, 2024

Thank you @Spomky.

@fabpot fabpot merged commit 09f9eb7 into symfony:7.2 Jul 10, 2024
8 of 10 checks passed
@derrabus
Copy link
Member

Would it make sense to apply this change to 6.4?

@derrabus
Copy link
Member

Ah, 6.4 is not affected. Nevermind. 🙈

@Spomky Spomky deleted the web-token-4 branch July 10, 2024 07:12
thibaut22200 added a commit to thibaut22200/symfony that referenced this pull request Jul 24, 2024
thibaut22200 added a commit to thibaut22200/symfony that referenced this pull request Jul 24, 2024
thibaut22200 added a commit to thibaut22200/symfony that referenced this pull request Jul 25, 2024
thibaut22200 added a commit to thibaut22200/symfony that referenced this pull request Jul 25, 2024
@fabpot fabpot mentioned this pull request Oct 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants