From 09d38468926feb01733e3a9ded15ca5fa2fdc1cf Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jaime=20Pe=CC=81rez=20Crespo?= <jaime.perez@uninett.no>
Date: Fri, 11 Aug 2017 09:36:40 +0200
Subject: [PATCH] bugfix: Make the core:TargetedID authproc filter return also
 a NameID object instead of its string representation.

---
 modules/core/lib/Auth/Process/TargetedID.php | 11 +++--------
 1 file changed, 3 insertions(+), 8 deletions(-)

diff --git a/modules/core/lib/Auth/Process/TargetedID.php b/modules/core/lib/Auth/Process/TargetedID.php
index 8d093e0b4..53d6cd77b 100644
--- a/modules/core/lib/Auth/Process/TargetedID.php
+++ b/modules/core/lib/Auth/Process/TargetedID.php
@@ -134,16 +134,11 @@ class sspmod_core_Auth_Process_TargetedID extends SimpleSAML_Auth_ProcessingFilt
 			if (isset($state['Destination']['entityid'])) {
 				$nameId->SPNameQualifier = $state['Destination']['entityid'];
 			}
-
-			$doc = \SAML2\DOMDocumentFactory::create();
-			$root = $doc->createElement('root');
-			$doc->appendChild($root);
-
-			$nameId->toXML($root);
-			$uid = $doc->saveXML($root->firstChild);
+		} else {
+			$nameId = $uid;
 		}
 
-		$state['Attributes']['eduPersonTargetedID'] = array($uid);
+		$state['Attributes']['eduPersonTargetedID'] = array($nameId);
 	}
 
 
-- 
GitLab