[PR #3729] [MERGED] move advanced_config section of /app/templates/_location.conf to top of default config #3750

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

📋 Pull Request Information

Original PR: https://github.com/NginxProxyManager/nginx-proxy-manager/pull/3729
Author: @clhey
Created: 4/28/2024
Status: Merged
Merged: 5/20/2024
Merged by: @jc21

Base: developHead: custom_proxy


📝 Commits (1)

  • 280bac8 advanced config move to top of default confg

📊 Changes

1 file changed (+2 additions, -3 deletions)

View changed files

📝 backend/templates/_location.conf (+2 -3)

📄 Description

I found that custom configurations cannot override the default configurations because nginx prioritizes the configurations appearing earlier. Therefore, I moved the advanced_config section of /app/templates/_location.conf to top of default config。

config:
image

before nginx proxy file:
image

after nginx proxy file:
image

see this issue:
https://github.com/NginxProxyManager/nginx-proxy-manager/issues/2675#issuecomment-2081407884


🔄 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/3729 **Author:** [@clhey](https://github.com/clhey) **Created:** 4/28/2024 **Status:** ✅ Merged **Merged:** 5/20/2024 **Merged by:** [@jc21](https://github.com/jc21) **Base:** `develop` ← **Head:** `custom_proxy` --- ### 📝 Commits (1) - [`280bac8`](https://github.com/NginxProxyManager/nginx-proxy-manager/commit/280bac8b430a3465f1fbc4d7e8d94e148ad740e7) advanced config move to top of default confg ### 📊 Changes **1 file changed** (+2 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `backend/templates/_location.conf` (+2 -3) </details> ### 📄 Description I found that custom configurations cannot override the default configurations because nginx prioritizes the configurations appearing earlier. Therefore, I moved the advanced_config section of /app/templates/_location.conf to top of default config。 config: ![image](https://github.com/NginxProxyManager/nginx-proxy-manager/assets/7664669/8dc8dcce-9394-4cf2-b17f-4cb256b90b9a) before nginx proxy file: ![image](https://github.com/NginxProxyManager/nginx-proxy-manager/assets/7664669/d225b7fa-a4ab-49b4-9170-f8d698d0f74f) after nginx proxy file: ![image](https://github.com/NginxProxyManager/nginx-proxy-manager/assets/7664669/a9956be6-1c52-40df-b38a-ac0f8ffc5843) see this issue: https://github.com/NginxProxyManager/nginx-proxy-manager/issues/2675#issuecomment-2081407884 --- <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:48 +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#3750
No description provided.