diff --git a/modules/metaedit/www/edit.php b/modules/metaedit/www/edit.php index 53023b1149e21846cb2b3655e0c4c27facb4ce7c..ea110f57e3298b0a31cf205d683ea6d4281ccaf9 100644 --- a/modules/metaedit/www/edit.php +++ b/modules/metaedit/www/edit.php @@ -2,7 +2,6 @@ /* Load simpleSAMLphp, configuration and metadata */ $config = SimpleSAML_Configuration::getInstance(); -$session = SimpleSAML_Session::getInstance(); $metaconfig = SimpleSAML_Configuration::getConfig('module_metaedit.php'); $mdh = new SimpleSAML_Metadata_MetaDataStorageHandlerSerialize($metaconfig->getValue('metahandlerConfig', NULL)); @@ -10,15 +9,13 @@ $mdh = new SimpleSAML_Metadata_MetaDataStorageHandlerSerialize($metaconfig->getV $authsource = $metaconfig->getValue('auth', 'login-admin'); $useridattr = $metaconfig->getValue('useridattr', 'eduPersonPrincipalName'); -if ($session->isValid($authsource)) { - $attributes = $session->getAttributes(); - // Check if userid exists - if (!isset($attributes[$useridattr])) - throw new Exception('User ID is missing'); - $userid = $attributes[$useridattr][0]; -} else { - SimpleSAML_Auth_Default::initLogin($authsource, SimpleSAML_Utilities::selfURL()); -} +$as = new SimpleSAML_Auth_Simple($authsource); +$as->requireAuth(); +$attributes = $as->getAttributes(); +// Check if userid exists +if (!isset($attributes[$useridattr])) + throw new Exception('User ID is missing'); +$userid = $attributes[$useridattr][0]; function requireOwnership($metadata, $userid) { if (!isset($metadata['owner'])) diff --git a/modules/metaedit/www/index.php b/modules/metaedit/www/index.php index bd2fb282722f9e0764c8d2dc113b7c661b81c6b9..f2e8d765ca6ba36c527e0fe6ab30b2033406f346 100644 --- a/modules/metaedit/www/index.php +++ b/modules/metaedit/www/index.php @@ -2,7 +2,6 @@ /* Load simpleSAMLphp, configuration and metadata */ $config = SimpleSAML_Configuration::getInstance(); -$session = SimpleSAML_Session::getInstance(); $metaconfig = SimpleSAML_Configuration::getConfig('module_metaedit.php'); $mdh = new SimpleSAML_Metadata_MetaDataStorageHandlerSerialize($metaconfig->getValue('metahandlerConfig', NULL)); @@ -10,15 +9,13 @@ $mdh = new SimpleSAML_Metadata_MetaDataStorageHandlerSerialize($metaconfig->getV $authsource = $metaconfig->getValue('auth', 'login-admin'); $useridattr = $metaconfig->getValue('useridattr', 'eduPersonPrincipalName'); -if ($session->isValid($authsource)) { - $attributes = $session->getAttributes(); - // Check if userid exists - if (!isset($attributes[$useridattr])) - throw new Exception('User ID is missing'); - $userid = $attributes[$useridattr][0]; -} else { - SimpleSAML_Auth_Default::initLogin($authsource, SimpleSAML_Utilities::selfURL()); -} +$as = new SimpleSAML_Auth_Simple($authsource); +$as->requireAuth(); +$attributes = $as->getAttributes(); +// Check if userid exists +if (!isset($attributes[$useridattr])) + throw new Exception('User ID is missing'); +$userid = $attributes[$useridattr][0]; function requireOwnership($metadata, $userid) { if (!isset($metadata['owner']))