Skip to content

[Security] add "anonymous: lazy" mode to firewalls #12390

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

Closed
chalasr opened this issue Sep 27, 2019 · 5 comments · Fixed by #13171
Closed

[Security] add "anonymous: lazy" mode to firewalls #12390

chalasr opened this issue Sep 27, 2019 · 5 comments · Fixed by #13171
Labels
Milestone

Comments

@chalasr
Copy link
Member

chalasr commented Sep 27, 2019

Q A
Feature PR symfony/symfony#33676
PR author(s) @nicolas-grekas
Merged in 4.4
@wouterj wouterj added this to the 4.4 milestone Sep 30, 2019
@javiereguiluz
Copy link
Member

Fixed in #13155.

@wouterj
Copy link
Member

wouterj commented Feb 17, 2020

I disagree with closing this as fixed, as the feature itself is not documented yet. Furthermore, I think we should also add the new AbstractListener with the support() method logic, in case someone implemented a custom authentication provider/listener and wants to support lazy firewalls.

@chalasr
Copy link
Member Author

chalasr commented Feb 17, 2020

@javiereguiluz We have number of support issues about what anonymous: lazy means, I would expect the fix to give an explanation about it

edit: missed the comment from @wouterj

@HeahDude HeahDude reopened this Feb 17, 2020
@javiereguiluz
Copy link
Member

You are right. I'm sorry I closed this one too quickly.

@HeahDude
Copy link
Contributor

I've opened #13171, let me know if we need to be more exhaustive or other places it should be mentioned. Thanks

@HeahDude HeahDude linked a pull request Feb 17, 2020 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants