{HttpFoundation] [Session] fixed session compatibility with memcached/redis session storage

Tom Avery · Tom Avery · commit eb8d8ebfc204 · 2013-08-16T16:48:21.000-04:00
Per https://bugs.php.net/bug.php?id=61470, and in fixing #7380, the following error occurs when using a Memcache or Redis session store w/ Symfony security: "Authentication exception occurred; redirecting to authentication entry point (A Token was not found in the SecurityContext.)". This patch applies the first fix only if the session store is "files" {HttpFoundation] [Session] fixed session compatibility with memcached/redis session storage Per https://bugs.php.net/bug.php?id=61470, and in fixing #7380, the following error occurs when using a Memcache or Redis session store w/ Symfony security: "Authentication exception occurred; redirecting to authentication entry point (A Token was not found in the SecurityContext.)". This patch applies the first fix only if the session store is "files"
diff --git a/src/Symfony/Component/HttpFoundation/Session/Storage/NativeSessionStorage.php b/src/Symfony/Component/HttpFoundation/Session/Storage/NativeSessionStorage.php
@@ -210,13 +210,15 @@ public function regenerate($destroy = false, $lifetime = null)
         $ret = session_regenerate_id($destroy);
 
         // workaround for https://bugs.php.net/bug.php?id=61470 as suggested by David Grudl
-        session_write_close();
-        if (isset($_SESSION)) {
-            $backup = $_SESSION;
-            session_start();
-            $_SESSION = $backup;
-        } else {
-            session_start();
+        if($this->getSaveHandler()->getSaveHandlerName() === 'files') {
+            session_write_close();
+            if (isset($_SESSION)) {
+                $backup = $_SESSION;
+                session_start();
+                $_SESSION = $backup;
+            } else {
+                session_start();
+            }
         }
 
         return $ret;
