[PR #5387] [CLOSED] Add Dual-Method Group Membership Verification for LDAP Integration #6487

New Issue

OVERLORD · 2026-02-05T10:33:36+03:00

OVERLORD commented

2026-02-05 10:33:36 +03:00

📋 Pull Request Information

Original PR: https://github.com/BookStackApp/BookStack/pull/5387
Author: @ghost
Created: 1/3/2025
Status: ❌ Closed

Base: development ← Head: LDAP

📝 Commits (1)

b767c42 LdapService.php ve package-lock.json dosyasındaki değişiklikler

📊 Changes

2 files changed (+9 additions, -2 deletions)

View changed files

📝 app/Access/LdapService.php (+8 -0)
📝 package-lock.json (+1 -2)

📄 Description

This PR introduces an enhancement to BookStack’s LDAP integration by adding support for a dual-method verification system to handle group membership checks. Currently, BookStack relies solely on the "memberOf" attribute to determine group memberships, which works well for LDAP systems like Microsoft Active Directory but poses challenges for systems like OpenLDAP that do not support "memberOf" by default.

_{🔄 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/5387 **Author:** [@ghost](https://github.com/ghost) **Created:** 1/3/2025 **Status:** ❌ Closed **Base:** `development` ← **Head:** `LDAP` --- ### 📝 Commits (1) - [`b767c42`](https://github.com/BookStackApp/BookStack/commit/b767c42ccf6a7c9aeca24a3aea88d99250d092e2) LdapService.php ve package-lock.json dosyasındaki değişiklikler ### 📊 Changes **2 files changed** (+9 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `app/Access/LdapService.php` (+8 -0) 📝 `package-lock.json` (+1 -2) </details> ### 📄 Description This PR introduces an enhancement to BookStack’s LDAP integration by adding support for a dual-method verification system to handle group membership checks. Currently, BookStack relies solely on the "memberOf" attribute to determine group memberships, which works well for LDAP systems like Microsoft Active Directory but poses challenges for systems like OpenLDAP that do not support "memberOf" by default. --- <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:33:36 +03:00

OVERLORD closed this issue

2026-02-05 10:33:38 +03:00

Sign in to join this conversation.

Branches Tags

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/BookStack#6487