Skip to content
Snippets Groups Projects
Commit daf235a2 authored by Tim van Dijen's avatar Tim van Dijen Committed by Tim van Dijen
Browse files

Fix ArtifactResolution #1343 (#1344)

Fix ArtifactResolution due to misuse of Issuer
parent 61983960
No related branches found
No related tags found
No related merge requests found
...@@ -47,6 +47,10 @@ if (!($request instanceof \SAML2\ArtifactResolve)) { ...@@ -47,6 +47,10 @@ if (!($request instanceof \SAML2\ArtifactResolve)) {
} }
$issuer = $request->getIssuer(); $issuer = $request->getIssuer();
if (!is_string($issuer)) {
$issuer = $issuer->getValue();
}
$spMetadata = $metadata->getMetaDataConfig($issuer, 'saml20-sp-remote'); $spMetadata = $metadata->getMetaDataConfig($issuer, 'saml20-sp-remote');
$artifact = $request->getArtifact(); $artifact = $request->getArtifact();
...@@ -62,7 +66,11 @@ if ($responseData !== null) { ...@@ -62,7 +66,11 @@ if ($responseData !== null) {
} }
$artifactResponse = new \SAML2\ArtifactResponse(); $artifactResponse = new \SAML2\ArtifactResponse();
$artifactResponse->setIssuer($idpEntityId);
$issuer = new \SAML2\XML\saml\Issuer();
$issuer->setValue($idpEntityId);
$artifactResponse->setIssuer($issuer);
$artifactResponse->setInResponseTo($request->getId()); $artifactResponse->setInResponseTo($request->getId());
$artifactResponse->setAny($responseXML); $artifactResponse->setAny($responseXML);
\SimpleSAML\Module\saml\Message::addSign($idpMetadata, $spMetadata, $artifactResponse); \SimpleSAML\Module\saml\Message::addSign($idpMetadata, $spMetadata, $artifactResponse);
......
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