Skip to content
Snippets Groups Projects
Unverified Commit b1b8c0f9 authored by Jaime Pérez Crespo's avatar Jaime Pérez Crespo
Browse files

bugfix: Fix instancing NoPassive errors in the IdP

The NoPassive constructor requirest the first parameter to be a constant indicating whether the error is due to the requester or the responder. Therefore, passing only an error message is not a valid way to create a NoPassive object anymore.
parent 519968c6
No related branches found
No related tags found
No related merge requests found
...@@ -2,6 +2,8 @@ ...@@ -2,6 +2,8 @@
namespace SimpleSAML; namespace SimpleSAML;
use SAML2\Constants as SAML2;
use SimpleSAML\Auth; use SimpleSAML\Auth;
use SimpleSAML\Error; use SimpleSAML\Error;
use SimpleSAML\Metadata\MetaDataStorageHandler; use SimpleSAML\Metadata\MetaDataStorageHandler;
...@@ -353,7 +355,7 @@ class IdP ...@@ -353,7 +355,7 @@ class IdP
private function authenticate(array &$state) private function authenticate(array &$state)
{ {
if (isset($state['isPassive']) && (bool) $state['isPassive']) { if (isset($state['isPassive']) && (bool) $state['isPassive']) {
throw new NoPassive('Passive authentication not supported.'); throw new NoPassive(SAML2::STATUS_RESPONDER, 'Passive authentication not supported.');
} }
$this->authSource->login($state); $this->authSource->login($state);
......
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