[PR #977] [MERGED] feat: manage nginx in another docker container #4259

New issue

Closed

opened 2026-02-28 11:59:43 +03:00 by kerem · 0 comments

kerem commented

2026-02-28 11:59:43 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/0xJacky/nginx-ui/pull/977
Author: @0xJacky
Created: 4/20/2025
Status: ✅ Merged
Merged: 4/21/2025
Merged by: @0xJacky

Base: dev ← Head: feat/docker-ui-only

📝 Commits (5)

d4a4ed1 feat(wip): docker ui only
a88f864 feat: add configuration file detection and sensitive content checks in code completion
e2c43be feat(ota): upgrade container
d079e80 chore: update docs
4aa42da refactor(preference): use pinia store

📊 Changes

90 files changed (+2363 additions, -572 deletions)

View changed files

📝 .devcontainer/docker-compose.yml (+23 -0)
📝 .devcontainer/init-nginx.sh (+1 -1)
📝 .devcontainer/start.sh (+1 -1)
📝 api/config/add.go (+6 -1)
📝 api/nginx/control.go (+21 -4)
📝 api/nginx/router.go (+1 -1)
📝 api/nginx/status.go (+6 -3)
📝 api/settings/settings.go (+1 -0)
📝 app/src/api/settings.ts (+1 -0)
➕ app/src/components/Breadcrumb/index.ts (+3 -0)
📝 app/src/components/CodeEditor/CodeCompletion.ts (+62 -3)
➕ app/src/components/EnvGroupTabs/index.ts (+3 -0)
➕ app/src/components/EnvIndicator/index.ts (+3 -0)
➕ app/src/components/ICP/index.ts (+3 -0)
➕ app/src/components/Logo/index.ts (+3 -0)
➕ app/src/components/NginxControl/index.ts (+3 -0)
➕ app/src/components/NodeSelector/index.ts (+3 -0)
➕ app/src/components/Notification/index.ts (+3 -0)
➕ app/src/components/OTPInput/index.ts (+3 -0)
➕ app/src/components/PageHeader/index.ts (+3 -0)

...and 70 more files

📄 Description

Close #234
Close #802
Close #856

Control the nginx in another container via the Go Docker client
If using Docker container, during OTA process, replace the container instead of the binary.
Update documentation about this feature.

_{🔄 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/0xJacky/nginx-ui/pull/977 **Author:** [@0xJacky](https://github.com/0xJacky) **Created:** 4/20/2025 **Status:** ✅ Merged **Merged:** 4/21/2025 **Merged by:** [@0xJacky](https://github.com/0xJacky) **Base:** `dev` ← **Head:** `feat/docker-ui-only` --- ### 📝 Commits (5) - [`d4a4ed1`](https://github.com/0xJacky/nginx-ui/commit/d4a4ed1e1c1397ae079860c9b18e815b5ae0f645) feat(wip): docker ui only - [`a88f864`](https://github.com/0xJacky/nginx-ui/commit/a88f8646e6821888dd3ec7e11287ce67293fe16a) feat: add configuration file detection and sensitive content checks in code completion - [`e2c43be`](https://github.com/0xJacky/nginx-ui/commit/e2c43be24adffe1f17a2d65b383b248cd23de57c) feat(ota): upgrade container - [`d079e80`](https://github.com/0xJacky/nginx-ui/commit/d079e805e5c81f97d8bab4ccfdb8dbdb5d5b968a) chore: update docs - [`4aa42da`](https://github.com/0xJacky/nginx-ui/commit/4aa42da642228e55b57cece1508f0de1104e6155) refactor(preference): use pinia store ### 📊 Changes **90 files changed** (+2363 additions, -572 deletions) <details> <summary>View changed files</summary> 📝 `.devcontainer/docker-compose.yml` (+23 -0) 📝 `.devcontainer/init-nginx.sh` (+1 -1) 📝 `.devcontainer/start.sh` (+1 -1) 📝 `api/config/add.go` (+6 -1) 📝 `api/nginx/control.go` (+21 -4) 📝 `api/nginx/router.go` (+1 -1) 📝 `api/nginx/status.go` (+6 -3) 📝 `api/settings/settings.go` (+1 -0) 📝 `app/src/api/settings.ts` (+1 -0) ➕ `app/src/components/Breadcrumb/index.ts` (+3 -0) 📝 `app/src/components/CodeEditor/CodeCompletion.ts` (+62 -3) ➕ `app/src/components/EnvGroupTabs/index.ts` (+3 -0) ➕ `app/src/components/EnvIndicator/index.ts` (+3 -0) ➕ `app/src/components/ICP/index.ts` (+3 -0) ➕ `app/src/components/Logo/index.ts` (+3 -0) ➕ `app/src/components/NginxControl/index.ts` (+3 -0) ➕ `app/src/components/NodeSelector/index.ts` (+3 -0) ➕ `app/src/components/Notification/index.ts` (+3 -0) ➕ `app/src/components/OTPInput/index.ts` (+3 -0) ➕ `app/src/components/PageHeader/index.ts` (+3 -0) _...and 70 more files_ </details> ### 📄 Description Close #234 Close #802 Close #856 - [x] Control the nginx in another container via the Go Docker client - [x] If using Docker container, during OTA process, replace the container instead of the binary. - [x] Update documentation about this feature. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>