Skip to content

Commit

Permalink
Bug: After renaming the PHP session variables to better match those u…
Browse files Browse the repository at this point in the history
…sed by the dbService, MyProxy certs no longer contained the correct extensions for ePPN and ePTID because of a case change.
  • Loading branch information
terrencegf committed Apr 3, 2020
1 parent e2eac2e commit d5face5
Showing 1 changed file with 12 additions and 1 deletion.
13 changes: 12 additions & 1 deletion src/Service/Skin.php
Expand Up @@ -465,10 +465,21 @@ public function setMyProxyInfo()
}

// Add eppn, eptid, open_id, and oidc if available
// Note that these values are lowercase after an update to make
// them the same as those used by the dbService. BUT, MyProxy
// expects the old versions. So this array maps the new lowercase
// versions back into the old ones.
$mpid = array(
'eppn' => 'ePPN',
'eptid' => 'ePTID',
'open_id' => 'openidID',
'oidc' => 'oidcID'
);
foreach (array('eppn','eptid','open_id','oidc') as $id) {
$sessvar = Util::getSessionVar($id);
if (strlen($sessvar) > 0) {
$infostr .= (strlen($infostr) > 0 ? ',' : '') . "$id=$sessvar";
$infostr .= (strlen($infostr) > 0 ? ',' : '') .
$mpid[$id] . "=" . $sessvar;
}
}

Expand Down

0 comments on commit d5face5

Please sign in to comment.