Skip to content
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

Remove SimplePreAuthenticatorInterface? #80

Closed
darthf1 opened this issue Sep 27, 2019 · 7 comments
Closed

Remove SimplePreAuthenticatorInterface? #80

darthf1 opened this issue Sep 27, 2019 · 7 comments
Milestone

Comments

@darthf1
Copy link
Contributor

darthf1 commented Sep 27, 2019

Thanks for merging #75!

Looking at the logs, I still get the following deprecation message:

The "Auth0\JWTAuthBundle\Security\JWTAuthenticator" class implements "Symfony\Component\Security\Http\Authentication\SimplePreAuthenticatorInterface" that is deprecated since Symfony 4.2, use Guard instead.

Since Guard is supported since SF 2.8, and the current latest supported release is 3.4.31, is it possible to remove the SimplePreAuth implementation alltogether?

@joshcanhelp
Copy link
Contributor

@darthf1 - Is that something you can do in your app or does it have to be removed from the library itself?

I'm fine with deprecating it now but it will need to be a major release to remove it completely.

@darthf1
Copy link
Contributor Author

darthf1 commented Dec 5, 2019

I think this can be closed as i have no issues with SF 5.0

@darthf1 darthf1 closed this as completed Dec 5, 2019
@joshcanhelp joshcanhelp added this to the 3.3.0 milestone Dec 5, 2019
@gigo6000
Copy link

gigo6000 commented Feb 25, 2020

@darthf1 I know this is old, but could you please share your solution? the JWTAuthenticator provided by the bundle is not supported by Symfony 5 because of this SimplePreAuthenticatorInterface. I'm trying to implement this bundle on a fresh Symfony 5 project because the composer.json allows it but I think is not really ready for that version.

@darthf1
Copy link
Contributor Author

darthf1 commented Feb 25, 2020

@gigo6000 I have "auth0/jwt-auth-bundle": "3.3.1", in my composer.json without any issues related to the SimplePreAuthenticatorInterface.

@gigo6000
Copy link

@darthf1 that's weird, I guess you're using a different authenticator, could you share your security.yml firewall config?

@gigo6000
Copy link

gigo6000 commented Feb 25, 2020

I realized there's actually a guard authenticator included in the bundle: https://github.com/auth0/jwt-auth-bundle/blob/master/src/Security/Guard/JwtGuardAuthenticator.php, and the service id is jwt_auth.security.guard.jwt_guard_authenticator, so the jwt_auth.jwt_authenticator should only be used for older Symfony versions.

@obukhov
Copy link

obukhov commented Feb 21, 2021

@gigo6000 thanks so much for your message. I wish that would be written in the doc for this package (which is missing)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants