Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

README.md has incorrect AcmeDemoBundle removal steps #434

Closed
noetix opened this Issue · 5 comments

4 participants

@noetix

Following the AcmeDemoBundle removal steps exactly has unexpected results.

The documentation says:

  • remove the security.providers, security.firewalls.login and security.firewalls.secured_area entries in the security.yml file or tweak the security configuration to fit your needs.

The first error we encounter:

[Symfony\Component\Config\Definition\Exception\InvalidConfigurationException]
The child node "providers" at path "security" must be configured.

This is because security.providers has to have at least one provider.

The second error we encounter, after we re-instate our security provider:

[InvalidArgumentException]
You must at least add one authentication provider.

Again, we need at least one "firewall".

Here is how the community is resolving this issue: http://stackoverflow.com/questions/13022393/symfony2-removing-the-acmebundle-demo-results-in-a-security-provider-error

@dlsniper

Imo we should have a way to disable the security bundle if we don't need it.
Removing it from AppKernel looks messy to me so maybe we need a configuration parameter to skip the initialization of all security stuff if we don't need it.

cc @fabpot @schmittjoh

@lsmith77
Collaborator

i dont understand why that would be considered messy? the SE is a starting point .. you can fork it but from then on you make it your own.

@dlsniper

Consider having this in security.yml instead of removing the file and the line from the AppKernel:

jms_security_extra:
    secure_all_services: false
    expressions: true

security:
    enabled: false

For me at least, this is why config files are for, configure things instead of removing them from different places.

@dlsniper

Also, removing SecurityBundle also means removing JMSSecurityExtraBundle and also means removing security.yml which, if you don't know from begining, will cause some frustration for new users.

Plus adding them back later on the project is not fun! :)

@fabpot
Owner

Closing this old issue now that I've tweaked the README about how to remove the security configuration of the Acme bundle.

@fabpot fabpot closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.