]> BookStack Code Mirror - bookstack/commitdiff
SAML: Set static type to pass static checks
authorDan Brown <redacted>
Mon, 10 Jun 2024 09:31:35 +0000 (10:31 +0100)
committerDan Brown <redacted>
Mon, 10 Jun 2024 09:31:35 +0000 (10:31 +0100)
Not totally clear if underlying code can actually return null, but
playing it safe to remain as-is for now for patch release.

app/Access/Saml2Service.php

index 664b77aba4510289e3eccc98b59ced6c77205a9d..bb7e9b572e2eaa1303a6f1761eb2e7c0b4798fda 100644 (file)
@@ -133,6 +133,7 @@ class Saml2Service
         // value so that the exact encoding format is matched when checking the signature.
         // This is primarily due to ADFS encoding query params with lowercase percent encoding while
         // PHP (And most other sensible providers) standardise on uppercase.
+        /** @var ?string $samlRedirect */
         $samlRedirect = $toolkit->processSLO(true, $requestId, true, null, true);
         $errors = $toolkit->getErrors();