Skip to content

Commit

Permalink
SAML: Set static type to pass static checks
Browse files Browse the repository at this point in the history
Not totally clear if underlying code can actually return null, but
playing it safe to remain as-is for now for patch release.
  • Loading branch information
ssddanbrown committed Jun 10, 2024
1 parent d9b9e6c commit 11a7ccc
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions app/Access/Saml2Service.php
Original file line number Diff line number Diff line change
Expand Up @@ -133,6 +133,7 @@ public function processSlsResponse(?string $requestId): string
// 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();

Expand Down

0 comments on commit 11a7ccc

Please sign in to comment.