[PR #2512] [CLOSED] Add the option to use ldap:// with StartTLS #6010

New Issue

OVERLORD · 2026-02-05T10:22:29+03:00

OVERLORD commented

2026-02-05 10:22:29 +03:00

📋 Pull Request Information

Original PR: https://github.com/BookStackApp/BookStack/pull/2512
Author: @lahx
Created: 1/27/2021
Status: ❌ Closed

Base: master ← Head: master

📝 Commits (1)

db90935 Add the option to use ldap:// with StartTLS

📊 Changes

4 files changed (+17 additions, -0 deletions)

View changed files

📝 .env.example.complete (+1 -0)
📝 app/Auth/Access/Ldap.php (+10 -0)
📝 app/Auth/Access/LdapService.php (+5 -0)
📝 app/Config/services.php (+1 -0)

📄 Description

ldaps:// seems to be depreacted and ldap:// with StartTLS is the preferred way.
In case a ldap server makes the use of StartTLS mandatory (-ZZ option on cli) php needs to execute ldap_start_tls() on the connection.

_{🔄 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/2512 **Author:** [@lahx](https://github.com/lahx) **Created:** 1/27/2021 **Status:** ❌ Closed **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (1) - [`db90935`](https://github.com/BookStackApp/BookStack/commit/db90935bdb389293bacc3309682be837f39e3ff1) Add the option to use ldap:// with StartTLS ### 📊 Changes **4 files changed** (+17 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `.env.example.complete` (+1 -0) 📝 `app/Auth/Access/Ldap.php` (+10 -0) 📝 `app/Auth/Access/LdapService.php` (+5 -0) 📝 `app/Config/services.php` (+1 -0) </details> ### 📄 Description ldaps:// seems to be depreacted and ldap:// with StartTLS is the preferred way. In case a ldap server makes the use of StartTLS mandatory (-ZZ option on cli) php needs to execute ldap_start_tls() on the connection. --- <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:22:29 +03:00

OVERLORD closed this issue

2026-02-05 10:22:30 +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#6010