Merge pull request #1092 from foerster-finsternis/master

CarlSchwan · web-flow · commit a665880c6eff · 2026-04-30T13:09:42.000+02:00
Fix logout condition based on last error reason
diff --git a/lib/Controller/SAMLController.php b/lib/Controller/SAMLController.php
@@ -501,7 +501,7 @@ public function singleLogoutService(): Http\RedirectResponse {
 					$this->userSession->logout();
 				}
 			}
-			if ($targetUrl !== '' && $targetUrl !== null && $auth && $auth->getLastErrorReason() !== null) {
+			if ($targetUrl !== '' && $targetUrl !== null && $auth && $auth->getLastErrorReason() === null) {
 				$this->userSession->logout();
 			}
 		}

Original file line number	Diff line number	Diff line change
`@@ -501,7 +501,7 @@ public function singleLogoutService(): Http\RedirectResponse {`
`501`	`501`	`$this->userSession->logout();`
`502`	`502`	`}`
`503`	`503`	`}`
`504`		`- if ($targetUrl !== '' && $targetUrl !== null && $auth && $auth->getLastErrorReason() !== null) {`
	`504`	`+ if ($targetUrl !== '' && $targetUrl !== null && $auth && $auth->getLastErrorReason() === null) {`
`505`	`505`	`$this->userSession->logout();`
`506`	`506`	`}`
`507`	`507`	`}`