[PR #2376] [MERGED] LDAP: Added TLS support #5989

New Issue

OVERLORD · 2026-02-05T10:21:56+03:00

OVERLORD commented

2026-02-05 10:21:56 +03:00

📋 Pull Request Information

Original PR: https://github.com/BookStackApp/BookStack/pull/2376
Author: @nicolas-rdgs
Created: 11/18/2020
Status: ✅ Merged
Merged: 2/7/2021
Merged by: @ssddanbrown

Base: master ← Head: master

📝 Commits (1)

20f9a50 LDAP: Added TLS support

📊 Changes

3 files changed (+17 additions, -6 deletions)

View changed files

📝 .env.example.complete (+1 -0)
📝 app/Auth/Access/LdapService.php (+15 -6)
📝 app/Config/services.php (+1 -0)

📄 Description

A new option is available in .env file: LDAP_START_TLS=true|false

It is necessary to add the option TLS_CACERT <path_to_bundle.pem> in /etc/openldap/ldap.conf to make it work. Because, unfortunately, the LDAP options is simply ignored by (my) PHP (i don't know why). Maybe yours it will work?

_{🔄 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/2376 **Author:** [@nicolas-rdgs](https://github.com/nicolas-rdgs) **Created:** 11/18/2020 **Status:** ✅ Merged **Merged:** 2/7/2021 **Merged by:** [@ssddanbrown](https://github.com/ssddanbrown) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (1) - [`20f9a50`](https://github.com/BookStackApp/BookStack/commit/20f9a50cee8ec22a20409d1abee26d5c33a1dabc) LDAP: Added TLS support ### 📊 Changes **3 files changed** (+17 additions, -6 deletions) <details> <summary>View changed files</summary> 📝 `.env.example.complete` (+1 -0) 📝 `app/Auth/Access/LdapService.php` (+15 -6) 📝 `app/Config/services.php` (+1 -0) </details> ### 📄 Description A new option is available in `.env` file: LDAP_START_TLS=true|false It is necessary to add the option `TLS_CACERT <path_to_bundle.pem>` in `/etc/openldap/ldap.conf` to make it work. Because, unfortunately, the LDAP options is simply ignored by (my) PHP (i don't know why). Maybe yours it will work? --- <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:21:56 +03:00

OVERLORD closed this issue

2026-02-05 10:21:58 +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#5989