Merge pull request #65 from eddmann/session-empty-string

Fix empty session value in Symfony adapter
brefphp · Sep 21, 2018 · 46e54ff · 46e54ff
2 parents 2d42ce7 + 5f5c681
commit 46e54ff
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/Bridge/Symfony/SymfonyAdapter.php b/src/Bridge/Symfony/SymfonyAdapter.php
@@ -48,14 +48,14 @@ public function handle(ServerRequestInterface $request): ResponseInterface
         return (new DiactorosFactory)->createResponse($symfonyResponse);
     }
 
-    private function loadSessionFromRequest(Request $symfonyRequest): ?string
+    private function loadSessionFromRequest(Request $symfonyRequest): string
     {
         if ($this->hasSessionsDisabled()) {
-            return null;
+            return '';
         }
 
         $this->httpKernel->getContainer()->get('session')->setId(
-            $sessionId = $symfonyRequest->cookies->get(session_name())
+            $sessionId = $symfonyRequest->cookies->get(session_name(), '')
         );
 
         return $sessionId;