bug #41486 [HttpKernel] Fix sessionListener when factory is injected without session (jderusse)

nicolas-grekas · nicolas-grekas · commit e2d1cb1d3ca5 · 2021-06-01T11:09:46.000+02:00
This PR was merged into the 5.4 branch. Discussion ---------- [HttpKernel] Fix sessionListener when factory is injected without session | Q | A | ------------- | --- | Branch? | 5.4 | Bug fix? | yes | New feature? | no | Deprecations? | no | Tickets | - | License | MIT | Doc PR | - Commits ------- b7e9194 Fix sessionListener when factory is injected without session
diff --git a/src/Symfony/Component/HttpKernel/EventListener/SessionListener.php b/src/Symfony/Component/HttpKernel/EventListener/SessionListener.php
@@ -38,7 +38,7 @@ public function onKernelRequest(RequestEvent $event)
     {
         parent::onKernelRequest($event);
 
-        if (!$event->isMainRequest() || !$this->container->has('session')) {
+        if (!$event->isMainRequest() || (!$this->container->has('session') && !$this->container->has('session_factory'))) {
             return;
         }
 

Original file line number	Diff line number	Diff line change
`@@ -38,7 +38,7 @@ public function onKernelRequest(RequestEvent $event)`
`38`	`38`	`{`
`39`	`39`	`parent::onKernelRequest($event);`
`40`	`40`
`41`		`- if (!$event->isMainRequest() \|\| !$this->container->has('session')) {`
	`41`	`+ if (!$event->isMainRequest() \|\| (!$this->container->has('session') && !$this->container->has('session_factory'))) {`
`42`	`42`	`return;`
`43`	`43`	`}`
`44`	`44`