bugfix: Fix a bug in AuthMemCookie that prevented the cookie from being set.

This was due to incorrect use of the SimpleSAML_SessionHandler::setCookie() method to set the cookie, instead of SimpleSAML\Utils\HTTP::setCookie().

lib/SimpleSAML/AuthMemCookie.php

@@ -145,8 +145,7 @@ class SimpleSAML_AuthMemCookie
         $memcache->delete($sessionID);
 
         // delete the session cookie
-        $sessionHandler = SimpleSAML_SessionHandler::getSessionHandler();
-        $sessionHandler->setCookie($cookieName, null);
+        \SimpleSAML\Utils\HTTP::setCookie($cookieName, null);
     }
 
 

www/authmemcookie.php

@@ -34,11 +34,8 @@ try {
 
     // generate session id and save it in a cookie
     $sessionID = SimpleSAML\Utils\Random::generateID();
-
     $cookieName = $amc->getCookieName();
-
-    $sessionHandler = SimpleSAML_SessionHandler::getSessionHandler();
-    $sessionHandler->setCookie($cookieName, $sessionID);
+    \SimpleSAML\Utils\HTTP::setCookie($cookieName, $sessionID);
 
     // generate the authentication information
     $attributes = $s->getAttributes();