[PR #3618] [CLOSED] Enable PROXY procotol for proxy hosts and streams #3732

New issue

Closed

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

kerem commented

2026-02-26 08:31:44 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/NginxProxyManager/nginx-proxy-manager/pull/3618
Author: @snordmann
Created: 3/10/2024
Status: ❌ Closed

Base: develop ← Head: enable-proxy-protocol

📝 Commits (9)

c6465a5 Enable PROXY procotol for proxy hosts
a92624d Fix eslint errors in migrations
72abe50 Adjust ProxyHostObject in api.swagger.json
da29cd2 Add support for PROXY protocol on streams
c39e580 Fix cypress integration tests
671817d Add docs on how to test the code
e677bfa Merge pull request #4073 from NginxProxyManager/develop
5084cb7 Merge pull request #4077 from NginxProxyManager/develop
7912119 Merge branch 'master' into enable-proxy-protocol

📊 Changes

21 files changed (+236 additions, -31 deletions)

View changed files

📝 README.md (+5 -0)
📝 backend/internal/nginx.js (+1 -0)
➕ backend/migrations/20240310085523_proxy_protocol.js (+41 -0)
➕ backend/migrations/20240310100432_proxy_protocol_streams.js (+41 -0)
📝 backend/templates/_listen.conf (+20 -11)
➕ backend/templates/_proxy_protocol.conf (+6 -0)
📝 backend/templates/proxy_host.conf (+1 -0)
📝 backend/templates/stream.conf (+13 -6)
📝 docker/Dockerfile (+1 -1)
📝 docker/dev/Dockerfile (+1 -1)
📝 docker/docker-compose.dev.yml (+2 -0)
📝 docs/src/advanced-config/index.md (+23 -0)
📝 docs/src/setup/index.md (+2 -0)
📝 frontend/js/app/nginx/proxy/form.ejs (+17 -1)
📝 frontend/js/app/nginx/proxy/form.js (+11 -0)
📝 frontend/js/app/nginx/stream/form.ejs (+16 -0)
📝 frontend/js/app/nginx/stream/form.js (+14 -0)
📝 frontend/js/i18n/messages.json (+4 -0)
📝 frontend/js/models/proxy-host.js (+2 -0)
📝 frontend/js/models/stream.js (+13 -11)

...and 1 more files

📄 Description

As I see not too much progress in https://github.com/NginxProxyManager/nginx-proxy-manager/pull/3537, I decided to copy most of the code and (try to) fix the tests.
In this PR I also included the code necessary to enable the PROXY protocol for Streams.

Thank you @jwklijnsma for the previous work

_{🔄 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/3618 **Author:** [@snordmann](https://github.com/snordmann) **Created:** 3/10/2024 **Status:** ❌ Closed **Base:** `develop` ← **Head:** `enable-proxy-protocol` --- ### 📝 Commits (9) - [`c6465a5`](https://github.com/NginxProxyManager/nginx-proxy-manager/commit/c6465a5090d27474d74f8985d71ab4c07e0a60d0) Enable PROXY procotol for proxy hosts - [`a92624d`](https://github.com/NginxProxyManager/nginx-proxy-manager/commit/a92624d0868c9c3e63d325405f7b0a64ab8a96b9) Fix eslint errors in migrations - [`72abe50`](https://github.com/NginxProxyManager/nginx-proxy-manager/commit/72abe5079984f9338688d1ea12fd7af582ae707c) Adjust ProxyHostObject in api.swagger.json - [`da29cd2`](https://github.com/NginxProxyManager/nginx-proxy-manager/commit/da29cd2e0e076d01df9aa8aeddcd59b68957ee65) Add support for PROXY protocol on streams - [`c39e580`](https://github.com/NginxProxyManager/nginx-proxy-manager/commit/c39e580e811d573dc4cbb846784b91d39ab23488) Fix cypress integration tests - [`671817d`](https://github.com/NginxProxyManager/nginx-proxy-manager/commit/671817d640deaf6ed9adc15bd0de2c8ae539e8a0) Add docs on how to test the code - [`e677bfa`](https://github.com/NginxProxyManager/nginx-proxy-manager/commit/e677bfa2e871839a5d86e6312d67c613476249ff) Merge pull request #4073 from NginxProxyManager/develop - [`5084cb7`](https://github.com/NginxProxyManager/nginx-proxy-manager/commit/5084cb7296af716bc2736a21ff38e931e64ec613) Merge pull request #4077 from NginxProxyManager/develop - [`7912119`](https://github.com/NginxProxyManager/nginx-proxy-manager/commit/7912119d14f35ccfc3221273cea6b6e338493975) Merge branch 'master' into enable-proxy-protocol ### 📊 Changes **21 files changed** (+236 additions, -31 deletions) <details> <summary>View changed files</summary> 📝 `README.md` (+5 -0) 📝 `backend/internal/nginx.js` (+1 -0) ➕ `backend/migrations/20240310085523_proxy_protocol.js` (+41 -0) ➕ `backend/migrations/20240310100432_proxy_protocol_streams.js` (+41 -0) 📝 `backend/templates/_listen.conf` (+20 -11) ➕ `backend/templates/_proxy_protocol.conf` (+6 -0) 📝 `backend/templates/proxy_host.conf` (+1 -0) 📝 `backend/templates/stream.conf` (+13 -6) 📝 `docker/Dockerfile` (+1 -1) 📝 `docker/dev/Dockerfile` (+1 -1) 📝 `docker/docker-compose.dev.yml` (+2 -0) 📝 `docs/src/advanced-config/index.md` (+23 -0) 📝 `docs/src/setup/index.md` (+2 -0) 📝 `frontend/js/app/nginx/proxy/form.ejs` (+17 -1) 📝 `frontend/js/app/nginx/proxy/form.js` (+11 -0) 📝 `frontend/js/app/nginx/stream/form.ejs` (+16 -0) 📝 `frontend/js/app/nginx/stream/form.js` (+14 -0) 📝 `frontend/js/i18n/messages.json` (+4 -0) 📝 `frontend/js/models/proxy-host.js` (+2 -0) 📝 `frontend/js/models/stream.js` (+13 -11) _...and 1 more files_ </details> ### 📄 Description As I see not too much progress in https://github.com/NginxProxyManager/nginx-proxy-manager/pull/3537, I decided to copy most of the code and (try to) fix the tests. In this PR I also included the code necessary to enable the PROXY protocol for Streams. Thank you @jwklijnsma for the previous work --- <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:44 +03:00

closed this issue
added the
pull-request
label

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#3732

No description provided.

Rows
Columns