[PR #2749] [MERGED] v2.10.0 #3618

Closed
opened 2026-02-26 08:31:19 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/NginxProxyManager/nginx-proxy-manager/pull/2749
Author: @jc21
Created: 3/26/2023
Status: Merged
Merged: 3/27/2023
Merged by: @jc21

Base: masterHead: develop


📝 Commits (10+)

  • dad3e1d Adds support to run processes as a user/group, defined
  • 2a07445 Refactor configuration
  • 60175e6 Updates for ci stack
  • ea61b15 don't zip log files anymore
  • c3469de Linting fixes
  • f8e6c8d Fix mistake with debug output
  • 4fdc80b Fix logical error with keys and mysql config
  • 8fdb8ac Update docs
  • 79fedfc Use consistent docker-compose file version in docs
  • 77eb618 Fix pip installs running as non-root user

📊 Changes

37 files changed (+971 additions, -896 deletions)

View changed files

📝 .version (+1 -1)
📝 Jenkinsfile (+6 -6)
📝 README.md (+4 -2)
📝 backend/app.js (+4 -3)
📝 backend/db.js (+20 -26)
📝 backend/index.js (+0 -87)
📝 backend/internal/certificate.js (+30 -18)
📝 backend/internal/nginx.js (+16 -16)
backend/lib/config.js (+184 -0)
📝 backend/lib/validator/index.js (+1 -1)
📝 backend/models/now_helper.js (+2 -2)
📝 backend/models/token.js (+14 -20)
📝 backend/package.json (+2 -3)
📝 backend/setup.js (+7 -66)
📝 backend/yarn.lock (+363 -395)
📝 docker/Dockerfile (+6 -10)
📝 docker/docker-compose.ci.yml (+20 -18)
📝 docker/docker-compose.dev.yml (+18 -12)
docker/rootfs/bin/common.sh (+29 -0)
docker/rootfs/bin/handle-ipv6-setting (+0 -46)

...and 17 more files

📄 Description

Main change: run services inside the docker as a particular user/group, and the ownership of data/letsencrypt would also belong to this user/group.

Defaults to 911:911, seems to be the standard for linuxservers.

Define with PUID and GUID env vars.


🔄 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/NginxProxyManager/nginx-proxy-manager/pull/2749 **Author:** [@jc21](https://github.com/jc21) **Created:** 3/26/2023 **Status:** ✅ Merged **Merged:** 3/27/2023 **Merged by:** [@jc21](https://github.com/jc21) **Base:** `master` ← **Head:** `develop` --- ### 📝 Commits (10+) - [`dad3e1d`](https://github.com/NginxProxyManager/nginx-proxy-manager/commit/dad3e1da7cba6a3304752eea5e0871811d04bcee) Adds support to run processes as a user/group, defined - [`2a07445`](https://github.com/NginxProxyManager/nginx-proxy-manager/commit/2a0744500503f44ce6658de93aea24cf177cbaeb) Refactor configuration - [`60175e6`](https://github.com/NginxProxyManager/nginx-proxy-manager/commit/60175e6d8c2a4eab8c37d9bfbb5274a4c4980148) Updates for ci stack - [`ea61b15`](https://github.com/NginxProxyManager/nginx-proxy-manager/commit/ea61b15a40ca0e422dc5a84a6f5ef18b7361b8c6) don't zip log files anymore - [`c3469de`](https://github.com/NginxProxyManager/nginx-proxy-manager/commit/c3469de61b984e2d6b55084cd253537d10296463) Linting fixes - [`f8e6c8d`](https://github.com/NginxProxyManager/nginx-proxy-manager/commit/f8e6c8d018cbfba7be58c647bea17f7caa8767f9) Fix mistake with debug output - [`4fdc80b`](https://github.com/NginxProxyManager/nginx-proxy-manager/commit/4fdc80be01bc21958b83f0dcc5c5efba5d37ce5d) Fix logical error with keys and mysql config - [`8fdb8ac`](https://github.com/NginxProxyManager/nginx-proxy-manager/commit/8fdb8ac8534c4c68f0733554476840a36fd8428e) Update docs - [`79fedfc`](https://github.com/NginxProxyManager/nginx-proxy-manager/commit/79fedfcea457cdee4398eeb3da91af92a999a9ee) Use consistent docker-compose file version in docs - [`77eb618`](https://github.com/NginxProxyManager/nginx-proxy-manager/commit/77eb618758d2a8b855cc6aad6107f651958a385c) Fix pip installs running as non-root user ### 📊 Changes **37 files changed** (+971 additions, -896 deletions) <details> <summary>View changed files</summary> 📝 `.version` (+1 -1) 📝 `Jenkinsfile` (+6 -6) 📝 `README.md` (+4 -2) 📝 `backend/app.js` (+4 -3) 📝 `backend/db.js` (+20 -26) 📝 `backend/index.js` (+0 -87) 📝 `backend/internal/certificate.js` (+30 -18) 📝 `backend/internal/nginx.js` (+16 -16) ➕ `backend/lib/config.js` (+184 -0) 📝 `backend/lib/validator/index.js` (+1 -1) 📝 `backend/models/now_helper.js` (+2 -2) 📝 `backend/models/token.js` (+14 -20) 📝 `backend/package.json` (+2 -3) 📝 `backend/setup.js` (+7 -66) 📝 `backend/yarn.lock` (+363 -395) 📝 `docker/Dockerfile` (+6 -10) 📝 `docker/docker-compose.ci.yml` (+20 -18) 📝 `docker/docker-compose.dev.yml` (+18 -12) ➕ `docker/rootfs/bin/common.sh` (+29 -0) ➖ `docker/rootfs/bin/handle-ipv6-setting` (+0 -46) _...and 17 more files_ </details> ### 📄 Description Main change: run services inside the docker as a particular user/group, and the ownership of data/letsencrypt would also belong to this user/group. Defaults to 911:911, seems to be the standard for linuxservers. Define with `PUID` and `GUID` env vars. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-26 08:31:19 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/nginx-proxy-manager-NginxProxyManager#3618
No description provided.