[PR #751] [MERGED] Update config with clearer instructions for setting listen address in a docker setup #929

New issue

Closed

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

kerem commented

2026-02-27 09:10:17 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/lldap/lldap/pull/751
Author: @agrmohit
Created: 12/3/2023
Status: ✅ Merged
Merged: 12/3/2023
Merged by: @nitnelave

Base: main ← Head: patch-1

📝 Commits (1)

c2f379e docs(config): clarify docker networking setup

📊 Changes

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

View changed files

📝 lldap_config.docker_template.toml (+6 -2)

📄 Description

Updated the config template to make it clear that the user should not set LLDAP server to listen on localhost if it is running in a docker container as it would then not be able to accept connections originating from outside the container.

_{🔄 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/751 **Author:** [@agrmohit](https://github.com/agrmohit) **Created:** 12/3/2023 **Status:** ✅ Merged **Merged:** 12/3/2023 **Merged by:** [@nitnelave](https://github.com/nitnelave) **Base:** `main` ← **Head:** `patch-1` --- ### 📝 Commits (1) - [`c2f379e`](https://github.com/lldap/lldap/commit/c2f379e51cf50773d6b77ad3da8d279c24966eff) docs(config): clarify docker networking setup ### 📊 Changes **1 file changed** (+6 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `lldap_config.docker_template.toml` (+6 -2) </details> ### 📄 Description Updated the config template to make it clear that the user should not set LLDAP server to listen on localhost if it is running in a docker container as it would then not be able to accept connections originating from outside the container. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>