Fixed Anonymous Name Cookie Issue when URL Routing Enabled (bug #823)

Geeklog-Core · Dec 1, 2017 · ba71400 · ba71400
1 parent 419e471
commit ba71400
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/system/classes/router.class.php b/system/classes/router.class.php
@@ -86,10 +86,11 @@ private static function proxy($url)
 
         // Add the current cookies.  Otherwise, session will be lost.
         foreach ($_COOKIE as $key => $value) {
-            $request->addCookie(
-                htmlspecialchars($key, ENT_QUOTES, 'utf-8'),
-                htmlspecialchars($value, ENT_QUOTES, 'utf-8')
-            );
+            try {
+                $request->addCookie(urlencode($key), urlencode($value));
+            } catch (HTTP_Request2_LogicException $e) {
+                COM_errorLog(__METHOD__ . ': invalid cookie detected. name = "' . $key . '" value = "' . $value . '"');
+            }
         }
 
         try {