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

Add info about security.firewall.context in pitfall #2172

Closed
wants to merge 22 commits into from

Conversation

morgen2009
Copy link
Contributor

Q A
Doc fix? yes
New docs? no
Applies to all
Fixed tickets

I spent some time for looking into the source code to understand, what does the security/firewalls/{id}/context parameter do. The user guide for SonataUserBundle has an example with this parameter, but I did not found any information. The links in search results are all about the service "security.context".

weaverryan and others added 22 commits January 15, 2013 08:29
[Cookbook/Form] Fixed jQuery bug
…note

Adding a note about finding more examples of working with DI definitions
…ple_order

Moving registering extension above loading config in DI code sample
Adding notes about Extension registration to the DI component
askAndConfirm should return user's input to properly work.
Documenting how we should use the versionadded tag - see https://groups....
[fix] about security/firewalls/{id}/context in pitfalls
@weaverryan
Copy link
Member

Hi Vladimir!

Thanks for this PR - I've patched your change in at sha: 712ca65 and even expanded on a few things at sha: 342c66e

Thanks!

@weaverryan weaverryan closed this Feb 24, 2013
wouterj added a commit that referenced this pull request Jul 2, 2016
…henticated (pasdeloup)

This PR was merged into the 2.7 branch.

Discussion
----------

Caution about impersonation not compatible with pre authenticated

Caution added to close Symfony issue #2172 according to @fabpot comment in the Symfony #19059 PR.
I also improved the similar caution about REMOTE_USER to make it global.

Commits
-------

b5354e5 fix pre Authenticated -> pre-authenticated
f4249f3 improve caution about impersonation not compatible with pre authenticated firewalls
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

Successfully merging this pull request may close these issues.

5 participants