[PR #4467] [MERGED] OIDC: Add RP-initiated logout implementation #6358

Closed
opened 2026-02-05 10:30:14 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/BookStackApp/BookStack/pull/4467
Author: @joancyho
Created: 8/29/2023
Status: Merged
Merged: 12/7/2023
Merged by: @ssddanbrown

Base: developmentHead: fix/oidc-logout


📝 Commits (2)

📊 Changes

6 files changed (+78 additions, -0 deletions)

View changed files

📝 .env.example.complete (+4 -0)
📝 app/Access/Controllers/OidcController.php (+14 -0)
📝 app/Access/Oidc/OidcService.php (+39 -0)
📝 app/Config/oidc.php (+5 -0)
📝 resources/views/common/header.blade.php (+14 -0)
📝 routes/web.php (+2 -0)

📄 Description

Reference Issues/PRS: 3715

https://github.com/BookStackApp/BookStack/issues/3715

What does this implement/fix? Explain your changes.

If the authentication is through an OIDC provider, this change also logs the user out of the provider as well when the user clicks on "Logout" on Bookstack.

Please add the parameter OIDC_END_SESSION_ENDPOINT in .env file based on end_session_endpoint from /.well-known/openid-configuration
e.g.
OIDC_END_SESSION_ENDPOINT=

Any other comments?

No


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/BookStackApp/BookStack/pull/4467 **Author:** [@joancyho](https://github.com/joancyho) **Created:** 8/29/2023 **Status:** ✅ Merged **Merged:** 12/7/2023 **Merged by:** [@ssddanbrown](https://github.com/ssddanbrown) **Base:** `development` ← **Head:** `fix/oidc-logout` --- ### 📝 Commits (2) - [`6b55104`](https://github.com/BookStackApp/BookStack/commit/6b55104ecb0dfe8da154161c689d2a937d0ae0e3) Fixed OIDC Logout - [`a0942ef`](https://github.com/BookStackApp/BookStack/commit/a0942ef441045f07646c3ba06f5f804827efd992) Fixed OIDC Logout ### 📊 Changes **6 files changed** (+78 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `.env.example.complete` (+4 -0) 📝 `app/Access/Controllers/OidcController.php` (+14 -0) 📝 `app/Access/Oidc/OidcService.php` (+39 -0) 📝 `app/Config/oidc.php` (+5 -0) 📝 `resources/views/common/header.blade.php` (+14 -0) 📝 `routes/web.php` (+2 -0) </details> ### 📄 Description #### Reference Issues/PRS: 3715 https://github.com/BookStackApp/BookStack/issues/3715 #### What does this implement/fix? Explain your changes. If the authentication is through an OIDC provider, this change also logs the user out of the provider as well when the user clicks on "Logout" on Bookstack. Please add the parameter OIDC_END_SESSION_ENDPOINT in .env file based on end_session_endpoint from <issuer>/.well-known/openid-configuration e.g. OIDC_END_SESSION_ENDPOINT= #### Any other comments? No --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-05 10:30:14 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/BookStack#6358