-
Notifications
You must be signed in to change notification settings - Fork 450
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
Support authentication manager #685
base: master
Are you sure you want to change the base?
Conversation
Still working on this. Adding an authenticator that manages authentication via the new |
Not sure about the user provider dependency injection yet, will be testing in the context of another app. |
Token storage is already taken care of in |
@deguif I noticed when testing this in an app that an unauthenticated request results in an If you don't know off the top of your head, I can run the same test based on a different branch of our app. |
Does |
@iisisrael was this work finished in the end by chance? ... was super keen to transition to Symfony v6 but can't without the new authentication system :/ |
@victormacko yes, at least for this PR, or until changes are requested. Feel free to contribute by forking my fork, and I'll pull any changes into here. Some other Symfony 5.3 deprecation fixes were made on a fork off of this PR branch here. |
5ea0f0e
to
92756c0
Compare
Rebased to include commits "Remove support for Propel" and "Move Travis to GH Actions and fix CI builds". |
Removing support of Symfony 5.1 and 5.2 as a result of ProxyManager dependency change for ClassGenerator. |
Why is this not getting merged? |
Supports authentication listener creation when using Symfony's new authentication system.
I didn't see any need to add to the config documentation. Whether you have the above config enabled or not, it just works (see
SecurityExtension::createAuthenticationListeners()
).