[PR #236] [MERGED] Update Ldap.php to allow LDAP over SSL/TLS connections #5619

New Issue

OVERLORD · 2026-02-05T10:12:30+03:00

OVERLORD commented

2026-02-05 10:12:30 +03:00

📋 Pull Request Information

Original PR: https://github.com/BookStackApp/BookStack/pull/236
Author: @fredericmohr
Created: 11/15/2016
Status: ✅ Merged
Merged: 1/14/2017
Merged by: @ssddanbrown

Base: master ← Head: patch-2

📝 Commits (1)

340d3f8 Update Ldap.php

📊 Changes

1 file changed (+19 additions, -1 deletions)

View changed files

📝 app/Services/Ldap.php (+19 -1)

📄 Description

This is a very crude workaround to allow LDAPS connections. A better solution would be the one explained in the comments.

_{🔄 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/236 **Author:** [@fredericmohr](https://github.com/fredericmohr) **Created:** 11/15/2016 **Status:** ✅ Merged **Merged:** 1/14/2017 **Merged by:** [@ssddanbrown](https://github.com/ssddanbrown) **Base:** `master` ← **Head:** `patch-2` --- ### 📝 Commits (1) - [`340d3f8`](https://github.com/BookStackApp/BookStack/commit/340d3f833b5fd4a6ae7c5b383c26200da3ceb387) Update Ldap.php ### 📊 Changes **1 file changed** (+19 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `app/Services/Ldap.php` (+19 -1) </details> ### 📄 Description This is a very crude workaround to allow LDAPS connections. A better solution would be the one explained in the comments. --- <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:12:30 +03:00

OVERLORD closed this issue

2026-02-05 10:12:33 +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#5619