Skip to content

Commit

Permalink
Use assertion id for session index
Browse files Browse the repository at this point in the history
This was a privacy issue because this allows different SP to correlate
users, defeating persistent and transient NameID mechanisms.
#41
  • Loading branch information
pablothedude committed Jan 28, 2025
1 parent c72e17a commit de96373
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion library/EngineBlock/Corto/ProxyServer.php
Original file line number Diff line number Diff line change
Expand Up @@ -666,7 +666,7 @@ public function createEnhancedResponse(

// Copy over the Authentication information because the IdP did the authentication, not us.
$newAssertion->setAuthnInstant($sourceAssertion->getAuthnInstant());
$newAssertion->setSessionIndex($sourceAssertion->getSessionIndex());
$newAssertion->setSessionIndex($sourceAssertion->getId());

$newAssertion->setAuthnContextClassRef($sourceAssertion->getAuthnContextClassRef());
$newAssertion->setAuthnContextDeclRef($sourceAssertion->getAuthnContextDeclRef());
Expand Down

0 comments on commit de96373

Please sign in to comment.