[PR #3742] [CLOSED] Multiple LDAP servers #6254

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

📋 Pull Request Information

Original PR: https://github.com/BookStackApp/BookStack/pull/3742
Author: @Khazhinov
Created: 9/21/2022
Status: Closed

Base: developmentHead: development


📝 Commits (3)

  • 4bacc45 multiple ldap server
  • 965258b multiple ldap server update
  • 8658459 Merge branch 'BookStackApp:development' into development

📊 Changes

2 files changed (+56 additions, -18 deletions)

View changed files

📝 .env.example.complete (+1 -1)
📝 app/Auth/Access/LdapService.php (+55 -17)

📄 Description

Hello!
I want to express my gratitude, because Bookstack has become an indispensable tool for me in my work!
But in order to get a failover option for Bookstack service on a domain network, I needed to be able to specify multiple domain controller addresses.
I propose a simple implementation of this feature.

Just specify the LDAP_SERVER variable in the .env file in the format: dc1.domain.local:389;dc2.domain.local:389


🔄 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/3742 **Author:** [@Khazhinov](https://github.com/Khazhinov) **Created:** 9/21/2022 **Status:** ❌ Closed **Base:** `development` ← **Head:** `development` --- ### 📝 Commits (3) - [`4bacc45`](https://github.com/BookStackApp/BookStack/commit/4bacc45fb7daedb07830abaaf8399e17488f7cbf) multiple ldap server - [`965258b`](https://github.com/BookStackApp/BookStack/commit/965258baf5a73ac4a36b307d084ad8de837c7666) multiple ldap server update - [`8658459`](https://github.com/BookStackApp/BookStack/commit/86584591511f2c45d2ef9f1126eb8ffccbdfee06) Merge branch 'BookStackApp:development' into development ### 📊 Changes **2 files changed** (+56 additions, -18 deletions) <details> <summary>View changed files</summary> 📝 `.env.example.complete` (+1 -1) 📝 `app/Auth/Access/LdapService.php` (+55 -17) </details> ### 📄 Description Hello! I want to express my gratitude, because Bookstack has become an indispensable tool for me in my work! But in order to get a failover option for Bookstack service on a domain network, I needed to be able to specify multiple domain controller addresses. I propose a simple implementation of this feature. Just specify the ```LDAP_SERVER``` variable in the .env file in the format: ```dc1.domain.local:389;dc2.domain.local:389``` --- <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:27:48 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/BookStack#6254