[PR #1614] [MERGED] Feature: Add two new headers to proxy.conf #3466

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

📋 Pull Request Information

Original PR: https://github.com/NginxProxyManager/nginx-proxy-manager/pull/1614
Author: @the1ts
Created: 11/29/2021
Status: Merged
Merged: 1/2/2022
Merged by: @jc21

Base: developHead: feature/proxy-header-additions


📝 Commits (5)

  • 3dfe238 Add two new headers to proxy.conf
  • 2ac1026 Hack for python binary in newer node images
  • 5d106c4 Bump version
  • ab80fe1 Fixes custom certificate upload modal
  • 7800938 Update default.conf

📊 Changes

6 files changed (+12 additions, -8 deletions)

View changed files

📝 .version (+1 -1)
📝 Jenkinsfile (+1 -1)
📝 README.md (+1 -1)
📝 backend/templates/default.conf (+2 -2)
📝 docker/rootfs/etc/nginx/conf.d/include/proxy.conf (+2 -0)
📝 frontend/js/app/nginx/certificates/form.js (+5 -3)

📄 Description

Fixes #1609. Adding both X-Forwarded-Host and X-Forwarded-Port, this is vital for some services behind a proxy (used to allow creation of absolute links in html). I've had to include at least the Host version in the past for Jenkins and Nexus.
Been running locally for 24 hours, does not appear to break any of my 15+ services currently running behind NPM would allow people to host those problem services without the need for advanced configuration.


🔄 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/1614 **Author:** [@the1ts](https://github.com/the1ts) **Created:** 11/29/2021 **Status:** ✅ Merged **Merged:** 1/2/2022 **Merged by:** [@jc21](https://github.com/jc21) **Base:** `develop` ← **Head:** `feature/proxy-header-additions` --- ### 📝 Commits (5) - [`3dfe238`](https://github.com/NginxProxyManager/nginx-proxy-manager/commit/3dfe23836ce5616664b2d208dd6f3dd724a5e11e) Add two new headers to proxy.conf - [`2ac1026`](https://github.com/NginxProxyManager/nginx-proxy-manager/commit/2ac1026e4b2d4edf1f39dee3fb3bfed8585bfed1) Hack for python binary in newer node images - [`5d106c4`](https://github.com/NginxProxyManager/nginx-proxy-manager/commit/5d106c40640224a176712cbba882146dbdc4680b) Bump version - [`ab80fe1`](https://github.com/NginxProxyManager/nginx-proxy-manager/commit/ab80fe13e9968f6adbdc1783f71d08098a9f926d) Fixes custom certificate upload modal - [`7800938`](https://github.com/NginxProxyManager/nginx-proxy-manager/commit/7800938fd2ffa9e7faa1c4d286d0f7fb0d3516ba) Update default.conf ### 📊 Changes **6 files changed** (+12 additions, -8 deletions) <details> <summary>View changed files</summary> 📝 `.version` (+1 -1) 📝 `Jenkinsfile` (+1 -1) 📝 `README.md` (+1 -1) 📝 `backend/templates/default.conf` (+2 -2) 📝 `docker/rootfs/etc/nginx/conf.d/include/proxy.conf` (+2 -0) 📝 `frontend/js/app/nginx/certificates/form.js` (+5 -3) </details> ### 📄 Description Fixes #1609. Adding both X-Forwarded-Host and X-Forwarded-Port, this is vital for some services behind a proxy (used to allow creation of absolute links in html). I've had to include at least the Host version in the past for Jenkins and Nexus. Been running locally for 24 hours, does not appear to break any of my 15+ services currently running behind NPM would allow people to host those problem services without the need for advanced configuration. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-26 08:30:45 +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#3466
No description provided.