[PR #376] [MERGED] Enhance nginx ui docker image #4038

Closed
opened 2026-02-28 11:58:55 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/0xJacky/nginx-ui/pull/376
Author: @0xJacky
Created: 5/5/2024
Status: Merged
Merged: 5/8/2024
Merged by: @0xJacky

Base: devHead: docker/s6-overlay


📝 Commits (10+)

  • 9bd1507 feat(docker): integrate s6-overlay
  • 070c0b4 feat(docker): mkdir streams-enabled, streams-available
  • 45a6811 fix(docker): nginx restart always output network error
  • abc73bb enhance(docker): recreate access.log and error.log as regular files
  • 7669e34 chore(docs): update nginx log document
  • ee30e84 chore(demo): update dockerfile
  • f0dafa7 Update translation files
  • 9413978 fix: build ldflags
  • 2071891 fix: certificate selector can not select #375
  • d4fbeef chore(docs): update backend build command

📊 Changes

86 files changed (+5241 additions, -5171 deletions)

View changed files

📝 .gitignore (+1 -0)
📝 Dockerfile (+21 -10)
📝 README-es.md (+1 -1)
📝 README-vi_VN.md (+1 -1)
📝 README-zh_CN.md (+1 -1)
📝 README-zh_TW.md (+1 -1)
📝 README.md (+1 -1)
📝 api/cluster/environment.go (+37 -85)
📝 api/cluster/router.go (+1 -0)
📝 api/nginx/control.go (+19 -3)
📝 api/nginx/nginx.go (+43 -57)
📝 api/system/install.go (+2 -2)
📝 api/system/router.go (+1 -0)
📝 api/system/settings.go (+10 -20)
📝 app.example.ini (+30 -22)
📝 app/package.json (+13 -13)
📝 app/pnpm-lock.yaml (+3041 -3830)
📝 app/src/App.vue (+3 -0)
📝 app/src/api/environment.ts (+13 -1)
📝 app/src/api/ngx.ts (+1 -1)

...and 66 more files

📄 Description

  • fix: log viewer error when using docker #371 .
  • fix: close #312
  • feat: use s6-overlay to manage the services in docker #308 .
  • feat: use settings env to skip install #357 .
  • feat: use env to predefine admin user #214 .
  • feat: customize local environment name #313 .
  • feat: use settings file to predefine nodes #169 .
  • feat: reload cluster node settings from settings file. #169
  • chore: update documentation about this pr.

🔄 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/376 **Author:** [@0xJacky](https://github.com/0xJacky) **Created:** 5/5/2024 **Status:** ✅ Merged **Merged:** 5/8/2024 **Merged by:** [@0xJacky](https://github.com/0xJacky) **Base:** `dev` ← **Head:** `docker/s6-overlay` --- ### 📝 Commits (10+) - [`9bd1507`](https://github.com/0xJacky/nginx-ui/commit/9bd1507392060ffab9afeb39e73ad435d8c01d1c) feat(docker): integrate s6-overlay - [`070c0b4`](https://github.com/0xJacky/nginx-ui/commit/070c0b46204ecf3e77839024b471d4100185bfdf) feat(docker): mkdir streams-enabled, streams-available - [`45a6811`](https://github.com/0xJacky/nginx-ui/commit/45a68112b1ff7bddb3a5464e22dbe78aa317c27a) fix(docker): nginx restart always output network error - [`abc73bb`](https://github.com/0xJacky/nginx-ui/commit/abc73bbf77cd45751dd1ff0938776b774ddd041c) enhance(docker): recreate access.log and error.log as regular files - [`7669e34`](https://github.com/0xJacky/nginx-ui/commit/7669e34b62da14bfda0af59df688abe6c26a538d) chore(docs): update nginx log document - [`ee30e84`](https://github.com/0xJacky/nginx-ui/commit/ee30e840e3a2e2814cfc226db66c800ceb14bc41) chore(demo): update dockerfile - [`f0dafa7`](https://github.com/0xJacky/nginx-ui/commit/f0dafa79f2197d518dcb20da48a265d31e7c017c) Update translation files - [`9413978`](https://github.com/0xJacky/nginx-ui/commit/941397869739dae2e9166b6326d725d72fe49764) fix: build ldflags - [`2071891`](https://github.com/0xJacky/nginx-ui/commit/20718918412702df49816699ebe7716c037c1d5f) fix: certificate selector can not select #375 - [`d4fbeef`](https://github.com/0xJacky/nginx-ui/commit/d4fbeef498cc3e382cfb5f4edc3940483536c712) chore(docs): update backend build command ### 📊 Changes **86 files changed** (+5241 additions, -5171 deletions) <details> <summary>View changed files</summary> 📝 `.gitignore` (+1 -0) 📝 `Dockerfile` (+21 -10) 📝 `README-es.md` (+1 -1) 📝 `README-vi_VN.md` (+1 -1) 📝 `README-zh_CN.md` (+1 -1) 📝 `README-zh_TW.md` (+1 -1) 📝 `README.md` (+1 -1) 📝 `api/cluster/environment.go` (+37 -85) 📝 `api/cluster/router.go` (+1 -0) 📝 `api/nginx/control.go` (+19 -3) 📝 `api/nginx/nginx.go` (+43 -57) 📝 `api/system/install.go` (+2 -2) 📝 `api/system/router.go` (+1 -0) 📝 `api/system/settings.go` (+10 -20) 📝 `app.example.ini` (+30 -22) 📝 `app/package.json` (+13 -13) 📝 `app/pnpm-lock.yaml` (+3041 -3830) 📝 `app/src/App.vue` (+3 -0) 📝 `app/src/api/environment.ts` (+13 -1) 📝 `app/src/api/ngx.ts` (+1 -1) _...and 66 more files_ </details> ### 📄 Description - [x] fix: log viewer error when using docker #371 . - [x] fix: close #312 - [x] feat: use s6-overlay to manage the services in docker #308 . - [x] feat: use settings env to skip install #357 . - [x] feat: use env to predefine admin user #214 . - [x] feat: customize local environment name #313 . - [x] feat: use settings file to predefine nodes #169 . - [x] feat: reload cluster node settings from settings file. #169 - [x] chore: update documentation about this pr. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-28 11:58:55 +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-ui#4038
No description provided.