-
-
Notifications
You must be signed in to change notification settings - Fork 195
Issue with the "session" service #130
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
Comments
Doesn't the |
Yes, by default it is disabled (the Standard Edition enables is explicitly: https://github.com/symfony/symfony-standard/blob/3.3/app/config/config.yml#L26-L29). |
and the same is true for the Symfony Demo: https://github.com/symfony/symfony-demo/blob/master/app/config/config.yml#L45-L47 |
Thanks for the info. I'm closing this as "no need to fix it". But in symfony/symfony-docs#7809 I've added a note explaining that in 4.0, some important things (such as |
If HWIOAuthBundle needs a session, then we should have a recipe that enables sessions automatically (the same goes for other important bundles that need a session). |
The same happens while installing FOSUser in Symfony Flex (it requires an inexistent session service) |
@fabpot What would be the correct method to create a recipe which enables sessions? I could create a meta package for it which adds a config file to set the framework.session option but I guess this could easily conflict if it was already set up. edit: I see it's discussed here too #247 so let's keep it in there. |
When upgrading a Symfony app to 3.3.4 + Flex and using HWIOAuthBundle, I saw this error message:
The error message is correct, because the
session
service doesn't exist:The HttpFoundation 3.3.4 component is installed in the app, but the
session
service doesn't exist.For comparison, the Symfony Demo app, which uses the exact same 3.3.4 version, but doesn't use Flex, has the
session
service defined:The text was updated successfully, but these errors were encountered: