Skip to content

[symfony/security-bundle:5.3] Enable authenticator security by default #944

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.

Sign up for GitHub

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

Jump to bottom
Merged
1 commit merged into from
May 8, 2021
Merged

[symfony/security-bundle:5.3] Enable authenticator security by default #944

1 commit merged into from
May 8, 2021

Conversation

wouterj
Copy link
Member

@wouterj wouterj commented May 7, 2021

Q A
License MIT
Doc issue/PR I'm working on it 😉

As of 5.3, the authenticator security is no longer experimental (ref: symfony/symfony#40814 ). We're going to revamp the security docs for 5.3 to always use the authenticator security. I think it's good to also default to this new system in 5.3.

--- symfony/security-bundle/5.1/config/packages/security.yaml
+++ symfony/security-bundle/5.3/config/packages/security.yaml
@@ -1,4 +1,6 @@
 security:
+    # https://symfony.com/doc/current/security/experimental_authenticators.html
+    enable_authenticator_manager: true
     # https://symfony.com/doc/current/security.html#where-do-users-come-from-user-providers
     providers:
         users_in_memory: { memory: null }
@@ -7,7 +9,6 @@
             pattern: ^/(_(profiler|wdt)|css|images|js)/
             security: false
         main:
-            anonymous: true
             lazy: true
             provider: users_in_memory

ghost
ghost approved these changes May 7, 2021
View reviewed changes
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request passes validation.

@wouterj wouterj changed the title [SecurityBundle 5.3] Enable authenticator security by default [symfony/security-bundle:5.3] Enable authenticator security by default May 7, 2021
ghost
ghost approved these changes May 7, 2021
View reviewed changes
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request passes validation.

@ghost ghost merged commit e1863f9 into symfony:master May 8, 2021
@wouterj wouterj deleted the security-53 branch May 8, 2021 22:17
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@derrabus derrabus derrabus approved these changes

@chalasr chalasr chalasr approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@wouterj @derrabus @chalasr
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy