diff --git a/lib/SimpleSAML/XML/SAML20/AuthnRequest.php b/lib/SimpleSAML/XML/SAML20/AuthnRequest.php index bbef2dd6f93e1e3d526b7fc6c4592622d65e69f2..d7bf991be12f6fa5c757b9434654760e0ea6179c 100644 --- a/lib/SimpleSAML/XML/SAML20/AuthnRequest.php +++ b/lib/SimpleSAML/XML/SAML20/AuthnRequest.php @@ -142,9 +142,9 @@ class SimpleSAML_XML_SAML20_AuthnRequest { try{ return $this->isSamlBoolTrue($ispas); }catch(Exception $e){ - // ... I don't understand, default to false - return FALSE; - // throw new Exception('Invalid value of IsPassive attribute in SAML2 AuthnRequest.'); + // ... I don't understand ... + // return FALSE; + throw new Exception('Invalid value of IsPassive attribute in SAML2 AuthnRequest.'); } } @@ -171,9 +171,9 @@ class SimpleSAML_XML_SAML20_AuthnRequest { try{ return $this->isSamlBoolTrue($fa); } catch(Exception $e){ - // ... I don't understand, default to false - return FALSE; - // throw new Exception('Invalid value of ForceAuthn attribute in SAML2 AuthnRequest.'); + // ... I don't understand ... + // return FALSE; + throw new Exception('Invalid value of ForceAuthn attribute in SAML2 AuthnRequest.'); } } @@ -308,6 +308,7 @@ class SimpleSAML_XML_SAML20_AuthnRequest { * @return bool TRUE or FALSE */ private function isSamlBoolTrue($boolSaml){ + $boolSaml = strtolower($boolSaml); if($boolSaml === 'true' || $boolSaml === '1') { return TRUE; } elseif($boolSaml === 'false' || $boolSaml === '0') {