Skip to content

[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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

vincentchalamon
Copy link
Contributor

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 following RFC 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

@carsonbot
Copy link

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

@vincentchalamon vincentchalamon marked this pull request as ready for review May 21, 2022 17:45
@carsonbot carsonbot added this to the 6.2 milestone May 21, 2022
@vincentchalamon vincentchalamon force-pushed the feat/security/bearer-authenticator branch from 5fdb0d0 to a738763 Compare May 21, 2022 17:48
@lyrixx lyrixx changed the title feat: implement HttpBearerAuthenticator [Security] Implement HttpBearerAuthenticator May 23, 2022
@vincentchalamon
Copy link
Contributor Author

Closing in favor of #46428

@vincentchalamon vincentchalamon deleted the feat/security/bearer-authenticator branch July 28, 2022 11:14
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.

[Security] Add Bearer Authenticator
2 participants