Skip to content

Commit

Permalink
Fix integration test
Browse files Browse the repository at this point in the history
The Mock IdP must set a Destination attribute in the <Response>, as per simplesamlphp/saml2@b4daf21
  • Loading branch information
baszoetekouw committed Nov 23, 2024
1 parent 71ce882 commit 6b9e772
Showing 1 changed file with 4 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -99,6 +99,10 @@ public function singleSignOnAction(Request $request, $idpName)
$destination = ($authnRequest->getAssertionConsumerServiceURL() ?
$authnRequest->getAssertionConsumerServiceURL() :
$response->getDestination());
$response->getDestination());

/* set the destination element of the response to the ACS URL */
$response->setDestination($destination);

if ($mockIdp->mustUseHttpRedirect()) {
$redirect = new HTTPRedirect();
Expand Down

0 comments on commit 6b9e772

Please sign in to comment.