[PR #94] [MERGED] Fix issues with docker around users and permissions #540

New issue

Closed

opened 2026-02-27 08:17:55 +03:00 by kerem · 0 comments

kerem commented

2026-02-27 08:17:55 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/lldap/lldap/pull/94
Author: @nitnelave
Created: 11/27/2021
Status: ✅ Merged
Merged: 11/27/2021
Merged by: @nitnelave

Base: main ← Head: fix_config

📝 Commits (2)

3ebca01 config: Prevent loading the wrong server_key
7bf490c docker: Fix permission issues, remove user from container

📊 Changes

5 files changed (+46 additions, -33 deletions)

View changed files

📝 Dockerfile (+8 -18)
📝 README.md (+4 -0)
📝 docker-entrypoint.sh (+20 -0)
📝 server/src/infra/cli.rs (+5 -0)
📝 server/src/infra/configuration.rs (+9 -15)

📄 Description

By making the /app folder world-readable, we have nothing that ties us to the app user, so we can run the container as any user. This, and some added startup checks for permissions, should help with most issues that users experience.

Fixes #86

_{🔄 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/94 **Author:** [@nitnelave](https://github.com/nitnelave) **Created:** 11/27/2021 **Status:** ✅ Merged **Merged:** 11/27/2021 **Merged by:** [@nitnelave](https://github.com/nitnelave) **Base:** `main` ← **Head:** `fix_config` --- ### 📝 Commits (2) - [`3ebca01`](https://github.com/lldap/lldap/commit/3ebca01180f203043838e1a2414026038094f623) config: Prevent loading the wrong server_key - [`7bf490c`](https://github.com/lldap/lldap/commit/7bf490cb477ee32caa6ff6081f43b04aa515decd) docker: Fix permission issues, remove user from container ### 📊 Changes **5 files changed** (+46 additions, -33 deletions) <details> <summary>View changed files</summary> 📝 `Dockerfile` (+8 -18) 📝 `README.md` (+4 -0) 📝 `docker-entrypoint.sh` (+20 -0) 📝 `server/src/infra/cli.rs` (+5 -0) 📝 `server/src/infra/configuration.rs` (+9 -15) </details> ### 📄 Description By making the /app folder world-readable, we have nothing that ties us to the `app` user, so we can run the container as any user. This, and some added startup checks for permissions, should help with most issues that users experience. Fixes #86 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>