diff --git a/modules/saml/lib/Auth/Process/PersistentNameID2TargetedID.php b/modules/saml/lib/Auth/Process/PersistentNameID2TargetedID.php index b6db278e625bb1206760b0ab8053c1fd25b21b88..7d911300a03a759f8b4d763e90e1778328eb7be3 100644 --- a/modules/saml/lib/Auth/Process/PersistentNameID2TargetedID.php +++ b/modules/saml/lib/Auth/Process/PersistentNameID2TargetedID.php @@ -1,78 +1,83 @@ <?php + /** - * Authproc filter to create the eduPersonTargetedID attribute from the persistent NameID. + * Authentication processing filter to create the eduPersonTargetedID attribute from the persistent NameID. * - * @package simpleSAMLphp + * @package SimpleSAMLphp */ -class sspmod_saml_Auth_Process_PersistentNameID2TargetedID extends SimpleSAML_Auth_ProcessingFilter { - - /** - * The attribute we should save the NameID in. - * - * @var string - */ - private $attribute; +class sspmod_saml_Auth_Process_PersistentNameID2TargetedID extends SimpleSAML_Auth_ProcessingFilter +{ + /** + * The attribute we should save the NameID in. + * + * @var string + */ + private $attribute; - /** - * Whether we should insert it as an saml:NameID element. - * - * @var boolean - */ - private $nameId; + /** + * Whether we should insert it as an saml:NameID element. + * + * @var boolean + */ + private $nameId; - /** - * Initialize this filter, parse configuration. - * - * @param array $config Configuration information about this filter. - * @param mixed $reserved For future use. - */ - public function __construct($config, $reserved) { - parent::__construct($config, $reserved); - assert('is_array($config)'); - if (isset($config['attribute'])) { - $this->attribute = (string)$config['attribute']; - } else { - $this->attribute = 'eduPersonTargetedID'; - } + /** + * Initialize this filter, parse configuration. + * + * @param array $config Configuration information about this filter. + * @param mixed $reserved For future use. + */ + public function __construct($config, $reserved) + { + parent::__construct($config, $reserved); + assert('is_array($config)'); - if (isset($config['nameId'])) { - $this->nameId = (bool)$config['nameId']; - } else { - $this->nameId = TRUE; - } - } + if (isset($config['attribute'])) { + $this->attribute = (string) $config['attribute']; + } else { + $this->attribute = 'eduPersonTargetedID'; + } + if (isset($config['nameId'])) { + $this->nameId = (bool) $config['nameId']; + } else { + $this->nameId = true; + } + } - /** - * Store a NameID to attribute. - * - * @param array &$state The request state. - */ - public function process(&$state) { - assert('is_array($state)'); - if (!isset($state['saml:NameID'][SAML2_Const::NAMEID_PERSISTENT])) { - SimpleSAML_Logger::warning('Unable to generate eduPersonTargetedID because no persistent NameID was available.'); - return; - } + /** + * Store a NameID to attribute. + * + * @param array &$state The request state. + */ + public function process(&$state) + { + assert('is_array($state)'); - $nameID = $state['saml:NameID'][SAML2_Const::NAMEID_PERSISTENT]; + if (!isset($state['saml:NameID'][SAML2_Const::NAMEID_PERSISTENT])) { + SimpleSAML_Logger::warning( + 'Unable to generate eduPersonTargetedID because no persistent NameID was available.' + ); + return; + } - if ($this->nameId) { - $doc = new DOMDocument(); - $root = $doc->createElement('root'); - $doc->appendChild($root); - SAML2_Utils::addNameId($root, $nameID); - $value = $doc->saveXML($root->firstChild); - } else { - $value = $nameID['Value']; - } + $nameID = $state['saml:NameID'][SAML2_Const::NAMEID_PERSISTENT]; - $state['Attributes'][$this->attribute] = array($value); - } + if ($this->nameId) { + $doc = new DOMDocument(); + $root = $doc->createElement('root'); + $doc->appendChild($root); + SAML2_Utils::addNameId($root, $nameID); + $value = $doc->saveXML($root->firstChild); + } else { + $value = $nameID['Value']; + } + $state['Attributes'][$this->attribute] = array($value); + } }