[PR #477] [MERGED] Correctly set permission on `/app` path and `entrypoint` #766

New issue

Closed

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

kerem commented

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

Owner

📋 Pull Request Information

Original PR: https://github.com/lldap/lldap/pull/477
Author: @martadinata666
Created: 3/17/2023
Status: ✅ Merged
Merged: 3/17/2023
Merged by: @nitnelave

Base: main ← Head: app-perm

📝 Commits (1)

1430f49 Correctly set permission on /app path and entrypoint

📊 Changes

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

View changed files

📝 .github/workflows/Dockerfile.ci.alpine (+2 -2)

📄 Description

On alpine dockerfile got old $CONTAINERUSER rather than $USER, oversighted on last overhaul, and undetected because our docker-entrypoint.sh correct this on runtime. Then detected after run without docker-entrypoint.sh while supporting https://github.com/nitnelave/lldap/pull/475

_{🔄 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/477 **Author:** [@martadinata666](https://github.com/martadinata666) **Created:** 3/17/2023 **Status:** ✅ Merged **Merged:** 3/17/2023 **Merged by:** [@nitnelave](https://github.com/nitnelave) **Base:** `main` ← **Head:** `app-perm` --- ### 📝 Commits (1) - [`1430f49`](https://github.com/lldap/lldap/commit/1430f4910ed735354fab20fd3a607242b6891c8c) Correctly set permission on `/app` path and `entrypoint` ### 📊 Changes **1 file changed** (+2 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/Dockerfile.ci.alpine` (+2 -2) </details> ### 📄 Description On alpine dockerfile got old `$CONTAINERUSER` rather than $USER, oversighted on last overhaul, and undetected because our `docker-entrypoint.sh` correct this on runtime. Then detected after run without `docker-entrypoint.sh` while supporting https://github.com/nitnelave/lldap/pull/475 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>