[PR #461] [MERGED] Make it more obvious LDAPS is supported (#460) #756

New issue

Closed

opened 2026-02-27 09:09:37 +03:00 by kerem · 0 comments

kerem commented

2026-02-27 09:09:37 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/lldap/lldap/pull/461
Author: @mitchins
Created: 2/26/2023
Status: ✅ Merged
Merged: 6/14/2023
Merged by: @martadinata666

Base: main ← Head: main

📝 Commits (2)

26593f0 Make it more obvious LDAPS is supported
f0b1fe8 Merge branch 'main' into main

📊 Changes

1 file changed (+6 additions, -0 deletions)

View changed files

📝 README.md (+6 -0)

📄 Description

Update example compose config showing both port and alluding to the environment variable that controls the certificates
This will resolve #460 in that searching for "SSL" or "CERT" or "KEY" in the readme yields no results for those not familiar to search for "LDAPS".

It is hoped that anyone reading the main page will easily see that "ssl" (tls) or "certificates" are indeed supported.

If this needs updating or seems redundant, just let me know.

_{🔄 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/lldap/lldap/pull/461 **Author:** [@mitchins](https://github.com/mitchins) **Created:** 2/26/2023 **Status:** ✅ Merged **Merged:** 6/14/2023 **Merged by:** [@martadinata666](https://github.com/martadinata666) **Base:** `main` ← **Head:** `main` --- ### 📝 Commits (2) - [`26593f0`](https://github.com/lldap/lldap/commit/26593f04c0ee898b351b2fad61a623167b2b716d) Make it more obvious LDAPS is supported - [`f0b1fe8`](https://github.com/lldap/lldap/commit/f0b1fe8443d305b7b0058904b37404c951ff15a3) Merge branch 'main' into main ### 📊 Changes **1 file changed** (+6 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `README.md` (+6 -0) </details> ### 📄 Description Update example compose config showing both port and alluding to the environment variable that controls the certificates This will resolve #460 in that searching for "SSL" or "CERT" or "KEY" in the readme yields no results for those not familiar to search for "LDAPS". It is hoped that anyone reading the main page will easily see that "ssl" (tls) or "certificates" are indeed supported. If this needs updating or seems redundant, just let me know. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>