Skip to content
Snippets Groups Projects
Commit b8ae7841 authored by Jaime Pérez Crespo's avatar Jaime Pérez Crespo
Browse files

Merge pull request #108 from nickamon/SAML-SP-NameId-Not-Exported

Bug fix to include the subject nameid in the data returned from a SAML 1.0 SSO authentication.
parents b84f2393 480e30eb
No related branches found
No related tags found
No related merge requests found
...@@ -98,7 +98,7 @@ class SimpleSAML_Auth_Default { ...@@ -98,7 +98,7 @@ class SimpleSAML_Auth_Default {
} }
/* Add those that should always be included. */ /* Add those that should always be included. */
foreach (array('Attributes', 'Expire', 'LogoutState', 'AuthnInstant', 'RememberMe') as $a) { foreach (array('Attributes', 'Expire', 'LogoutState', 'AuthnInstant', 'RememberMe', 'saml:sp:NameID') as $a) {
if (isset($state[$a])) { if (isset($state[$a])) {
$persistentAuthState[$a] = $state[$a]; $persistentAuthState[$a] = $state[$a];
} }
......
...@@ -81,5 +81,7 @@ $logoutState = array( ...@@ -81,5 +81,7 @@ $logoutState = array(
); );
$state['LogoutState'] = $logoutState; $state['LogoutState'] = $logoutState;
$state['saml:sp:NameID'] = $response->getNameID();
$source->handleResponse($state, $responseIssuer, $attributes); $source->handleResponse($state, $responseIssuer, $attributes);
assert('FALSE'); assert('FALSE');
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment