Be strict when checking return values.

jaimeperez · jaimeperez · commit 5d69753a61b4 · 2018-03-02T15:44:21.000+01:00
diff --git a/src/SAML2/HTTPRedirect.php b/src/SAML2/HTTPRedirect.php
@@ -226,7 +226,7 @@ public static function validateSignature(array $data, XMLSecurityKey $key)
             $key = Utils::castKey($key, $sigAlg);
         }
 
-        if (!$key->verifySignature($query, $signature)) {
+        if ($key->verifySignature($query, $signature) !== 1) {
             throw new \Exception('Unable to validate signature on query string.');
         }
     }

Original file line number	Diff line number	Diff line change
`@@ -226,7 +226,7 @@ public static function validateSignature(array $data, XMLSecurityKey $key)`
`226`	`226`	`$key = Utils::castKey($key, $sigAlg);`
`227`	`227`	`}`
`228`	`228`
`229`		`- if (!$key->verifySignature($query, $signature)) {`
	`229`	`+ if ($key->verifySignature($query, $signature) !== 1) {`
`230`	`230`	`throw new \Exception('Unable to validate signature on query string.');`
`231`	`231`	`}`
`232`	`232`	`}`