bugfix: Make sure we can deal with attribute values that are not strings (i.e. DOMNodeList).

jaimeperez · jaimeperez · commit 54e4791a4af8 · 2016-08-23T11:40:58.000+02:00
This is due to the update of the SAML2 library, that caused several other bugs, mainly with attributes like eduPersonTargetedID, which should always be an SAML NameID.
diff --git a/www/authmemcookie.php b/www/authmemcookie.php
@@ -74,11 +74,21 @@
 
     // store the authentication data in the memcache server
     $data = '';
-    foreach ($authData as $n => $v) {
-        if (is_array($v)) {
-            $v = implode(':', $v);
+    foreach ($authData as $name => $values) {
+        if (is_array($values)) {
+            foreach ($values as $i => $value) {
+                if (!is_a($value, 'DOMNodeList')) {
+                    continue;
+                }
+                /* @var \DOMNodeList $value */
+                if ($value->length === 0) {
+                    continue;
+                }
+                $values[$i] = new \SAML2\XML\saml\AttributeValue($value->item(0)->parentNode);
+            }
+            $values = implode(':', $values);
         }
-        $data .= $n.'='.$v."\r\n";
+        $data .= $name.'='.$values."\r\n";
     }
 
     $memcache = $amc->getMemcache();
