[PR #10917] [MERGED] Update Patchmon default Nginx config (IPv6 and correct scheme) #9096

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

📋 Pull Request Information

Original PR: https://github.com/community-scripts/ProxmoxVE/pull/10917
Author: @christiaangoossens
Created: 1/18/2026
Status: Merged
Merged: 1/18/2026
Merged by: @tremor021

Base: mainHead: patch-1


📝 Commits (1)

  • 7973cdf Make sure Patchmon also works over IPv6 and if there is another reverse proxy in front

📊 Changes

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

View changed files

📝 install/patchmon-install.sh (+10 -3)

📄 Description

✍️ Description

Updates the default nginx config for the Patchmon LXC container to solve issues found in https://github.com/PatchMon/PatchMon/issues/266.

Changes:

  • Now listening on both IPv4 and IPv6 addresses of the container
  • Removed unnecessary server_name to help with IPv6
  • Use X-Forwarded-Proto when there is a reverse proxy in front (similar to proxy_add_x_forwarded_for that was already set), otherwise fallback to normal scheme

There is likely no need to update existing container configs, if users experience issues such as the Websocket connections always showing up as "WS" instead of "WSS" even if connected over HTTPS, they can likely perform these changes themselves.

Fixes https://github.com/PatchMon/PatchMon/issues/266 (example at https://github.com/PatchMon/PatchMon/issues/266#issuecomment-3476340195)

Prerequisites (X in brackets)

  • Self-review completed – Code follows project standards.
  • Tested thoroughly – Changes work as expected.
  • No security risks – No hardcoded secrets, unnecessary privilege escalations, or permission issues.

🛠️ Type of Change (X in brackets)

  • 🐞 Bug fix – Resolves an issue without breaking functionality.
  • New feature – Adds new, non-breaking functionality.
  • 💥 Breaking change – Alters existing functionality in a way that may require updates.
  • 🆕 New script – A fully functional and tested script or script set.
  • 🌍 Website update – Changes to website-related JSON files or metadata.
  • 🔧 Refactoring / Code Cleanup – Improves readability or maintainability without changing functionality.
  • 📝 Documentation update – Changes to README, AppName.md, CONTRIBUTING.md, or other docs.

🔄 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/community-scripts/ProxmoxVE/pull/10917 **Author:** [@christiaangoossens](https://github.com/christiaangoossens) **Created:** 1/18/2026 **Status:** ✅ Merged **Merged:** 1/18/2026 **Merged by:** [@tremor021](https://github.com/tremor021) **Base:** `main` ← **Head:** `patch-1` --- ### 📝 Commits (1) - [`7973cdf`](https://github.com/community-scripts/ProxmoxVE/commit/7973cdf92acd866dbf3f30a46a93f8a221686119) Make sure Patchmon also works over IPv6 and if there is another reverse proxy in front ### 📊 Changes **1 file changed** (+10 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `install/patchmon-install.sh` (+10 -3) </details> ### 📄 Description ## ✍️ Description Updates the default nginx config for the Patchmon LXC container to solve issues found in https://github.com/PatchMon/PatchMon/issues/266. Changes: - Now listening on both IPv4 and IPv6 addresses of the container - Removed unnecessary server_name to help with IPv6 - Use X-Forwarded-Proto when there is a reverse proxy in front (similar to proxy_add_x_forwarded_for that was already set), otherwise fallback to normal scheme There is likely no need to update existing container configs, if users experience issues such as the Websocket connections always showing up as "WS" instead of "WSS" even if connected over HTTPS, they can likely perform these changes themselves. ## 🔗 Related Issue Fixes https://github.com/PatchMon/PatchMon/issues/266 (example at https://github.com/PatchMon/PatchMon/issues/266#issuecomment-3476340195) ## ✅ Prerequisites (**X** in brackets) - [X] **Self-review completed** – Code follows project standards. - [X] **Tested thoroughly** – Changes work as expected. - [X] **No security risks** – No hardcoded secrets, unnecessary privilege escalations, or permission issues. --- ## 🛠️ Type of Change (**X** in brackets) - [ ] 🐞 **Bug fix** – Resolves an issue without breaking functionality. - [X] ✨ **New feature** – Adds new, non-breaking functionality. - [ ] 💥 **Breaking change** – Alters existing functionality in a way that may require updates. - [ ] 🆕 **New script** – A fully functional and tested script or script set. - [ ] 🌍 **Website update** – Changes to website-related JSON files or metadata. - [ ] 🔧 **Refactoring / Code Cleanup** – Improves readability or maintainability without changing functionality. - [ ] 📝 **Documentation update** – Changes to `README`, `AppName.md`, `CONTRIBUTING.md`, or other docs. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-26 16:35:08 +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/ProxmoxVE#9096
No description provided.