IBX-6207: Requests containing front controller script causes session-…

…not-found exception (#377)
ezsystems · Nov 3, 2023 · 0f4d6ab · 0f4d6ab
1 parent 8880cf8
commit 0f4d6ab
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 2 deletions.
diff --git a/eZ/Bundle/EzPublishCoreBundle/EventListener/SessionInitByPostListener.php b/eZ/Bundle/EzPublishCoreBundle/EventListener/SessionInitByPostListener.php
@@ -27,9 +27,12 @@ public static function getSubscribedEvents()
     public function onSiteAccessMatch(PostSiteAccessMatchEvent $event)
     {
         $request = $event->getRequest();
-        $session = $request->getSession();
+        $session = null;
+        if ($request->hasSession()) {
+            $session = $request->getSession();
+        }
 
-        if (!$session || $event->getRequestType() !== HttpKernelInterface::MAIN_REQUEST) {
+        if (null === $session || $event->getRequestType() !== HttpKernelInterface::MAIN_REQUEST) {
             return;
         }
 

diff --git a/eZ/Bundle/EzPublishCoreBundle/Tests/EventListener/SessionInitByPostListenerTest.php b/eZ/Bundle/EzPublishCoreBundle/Tests/EventListener/SessionInitByPostListenerTest.php
@@ -117,4 +117,12 @@ public function testOnSiteAccessMatchNewSessionName()
 
         $this->listener->onSiteAccessMatch($event);
     }
+
+    public function testOnSiteAccessMatchNoSession(): void
+    {
+        $request = new Request();
+
+        $event = new PostSiteAccessMatchEvent(new SiteAccess('test'), $request, HttpKernelInterface::MAIN_REQUEST);
+        $this->listener->onSiteAccessMatch($event);
+    }
 }