Skip to content

Commit 2150cb0

Browse files
committed
fix: Return a logout URL even when SAML SLO is not configured
Signed-off-by: Carl Schwan <carlschwan@kde.org>
1 parent f528da3 commit 2150cb0

2 files changed

Lines changed: 9 additions & 12 deletions

File tree

core/AppInfo/Application.php

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -93,5 +93,4 @@ public function register(IRegistrationContext $context): void {
9393
public function boot(IBootContext $context): void {
9494
// ...
9595
}
96-
9796
}

lib/private/NavigationManager.php

Lines changed: 9 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -309,17 +309,15 @@ private function init(bool $resolveClosures = true): void {
309309
}
310310

311311
$logoutUrl = \OC_User::getLogoutUrl($this->urlGenerator);
312-
if ($logoutUrl !== '') {
313-
// Logout
314-
$this->add([
315-
'type' => 'settings',
316-
'id' => 'logout',
317-
'order' => 99999,
318-
'href' => $logoutUrl,
319-
'name' => $l->t('Log out'),
320-
'icon' => $this->urlGenerator->imagePath('core', 'actions/logout.svg'),
321-
]);
322-
}
312+
// Logout
313+
$this->add([
314+
'type' => 'settings',
315+
'id' => 'logout',
316+
'order' => 99999,
317+
'href' => $logoutUrl,
318+
'name' => $l->t('Log out'),
319+
'icon' => $this->urlGenerator->imagePath('core', 'actions/logout.svg'),
320+
]);
323321

324322
if ($this->isSubadmin()) {
325323
// User management

0 commit comments

Comments
 (0)