search

vivekrajenderan / simplesamlphp

Automatically exported from code.google.com/p/simplesamlphp
Other
0 stars 0 forks source link

SimpleSAML_Auth_Simple->getAuthData('saml:sp:IdP') not working when using SAML 1.0 artifact binding #551

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
In a SimpleSAMLPHP based Service Provider I ran into an issue when using 
getAuthData('saml:sp:IdP') to get the Entity ID of the IdP used for 
authentication.  It worked for SAML 2.0 based IdPs but not SAML 1.1 ones.

The fix was to add;

   $state['saml:sp:IdP'] = $responseIssuer;
   $state['PersistentAuthData'][] = 'saml:sp:IdP';

before the call to $source->handleResponse($state, $responseIssuer, 
$attributes); at the end of /simplesamlphp/modules/saml/www/sp/saml1-acs.php"

Original issue reported on code.google.com by timothyj...@googlemail.com on 7 May 2013 at 11:01

Attachments:

GoogleCodeExporter commented 9 years ago 
Thank you for reporting this bug! I have committed a fix for it in r3240. (I 
chose a slightly different fix than the one you submitted -- I moved the code 
into handleResponse().)

Original comment by olavmrk@gmail.com on 22 May 2013 at 8:28