Skip to content

Firewall & Security Context error #2837

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
jbafford opened this issue Jul 20, 2013 · 2 comments
Closed

Firewall & Security Context error #2837

jbafford opened this issue Jul 20, 2013 · 2 comments

Comments

@jbafford
Copy link
Contributor

In components/security/firewall.rst, in the Firewall and Security Context section, there are (at least) two errors:

  1. use Symfony\Component\Security\SecurityContext should be Symfony\Component\Security\Core\SecurityContext

  2. new SecurityContext() fails; it is missing two required parameters. If this is intentional, then some documentation should be provided to that effect.

An alternative (and possibly more useful) solution might be to suggest getting the security context from the container (e.g. saying $container->get('security.context')).

@stof
Copy link
Member

stof commented Jul 20, 2013

Getting it from the container is not an option in the doc of the component

@weaverryan
Copy link
Member

Thanks @jbafford for the report and @xabbuh for the fix!

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

No branches or pull requests

3 participants