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

Fix type confusion

parent 5b0979a8
Branches
Tags v1.2.4
No related merge requests found
...@@ -82,10 +82,10 @@ if ($prevAuth !== null && $prevAuth['id'] === $response->getId() && $prevAuth['i ...@@ -82,10 +82,10 @@ if ($prevAuth !== null && $prevAuth['id'] === $response->getId() && $prevAuth['i
throw new \SimpleSAML\Error\Exception('Duplicate assertion received.'); throw new \SimpleSAML\Error\Exception('Duplicate assertion received.');
} }
$idpMetadata = []; $idpMetadata = null;
$state = null; $state = null;
$stateId = $response->getInResponseTo(); $stateId = $response->getInResponseTo();
if (!empty($stateId)) { if (!empty($stateId)) {
// this should be a response to a request we sent earlier // this should be a response to a request we sent earlier
try { try {
...@@ -131,7 +131,7 @@ if ($state) { ...@@ -131,7 +131,7 @@ if ($state) {
SimpleSAML\Logger::debug('Received SAML2 Response from '.var_export($issuer, true).'.'); SimpleSAML\Logger::debug('Received SAML2 Response from '.var_export($issuer, true).'.');
if (empty($idpMetadata)) { if (is_null($idpMetadata)) {
$idpMetadata = $source->getIdPmetadata($issuer); $idpMetadata = $source->getIdPmetadata($issuer);
} }
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment