NotificationsYou must be signed in to change notification settings
Fork9.6k
Star30.4k

[Security] Implement HttpBearerAuthenticator#46429

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Closed

vincentchalamon wants to merge1 commit intosymfony:6.2fromvincentchalamon:feat/security/bearer-authenticator

Closed

[Security] Implement HttpBearerAuthenticator#46429

vincentchalamon wants to merge1 commit intosymfony:6.2fromvincentchalamon:feat/security/bearer-authenticator

Conversation

Copy link

Contributor

vincentchalamon commentedMay 21, 2022

Q	A
Branch?	6.2
Bug fix?	no
New feature?	yes
Deprecations?	no
Tickets	Fix#45844
License	MIT
Doc PR	TO DO

A proposal implementation for a Bearer authenticator followingRFC 6750 with an AbstractBearerAuthenticator allowing to create its own Bearer authenticator.

This implementation is based on lcobucci/jwt, but there is a discussion about this dependency on#45844.

TODO:

migrate ConfigurationFactory to framework-bundle configuration
write doc
write tests
create abstraction layer over lcobucci/jwt

Copy link

carsonbot commentedMay 21, 2022

Hey!

To help keep things organized, we don't allow "Draft" pull requests. Could you please click the "ready for review" button or close this PR and open a new one when you are done?

Note that a pull request does not have to be "perfect" or "ready for merge" when you first open it. We just want it to be ready for a first review.

Cheers!

Carsonbot