[PR #1698] [CLOSED] docs/reverse-proxy: use standard map, correct X-Forwarded-Proto varia… #2261

Closed
opened 2025-12-29 03:20:27 +01:00 by adam · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/juanfont/headscale/pull/1698
Author: @SuperSandro2000
Created: 1/24/2024
Status: Closed

Base: mainHead: patch-1


📝 Commits (1)

  • 1f0a511 docs/reverse-proxy: use standard map, correct X-Forwarded-Proto variable for nginx

📊 Changes

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

View changed files

📝 docs/reverse-proxy.md (+2 -3)

📄 Description

…ble for nginx

I tested this on NixOS. After switching from postgres back to sqlite and with the standard websocket proxy config, everything started to work.

Initially the none standard map which differs from upstreams docs https://nginx.org/en/docs/http/websocket.html got me very confused and I thought this would be somehow required.

Also $http_x_forwarded_proto doesn't exist, NixOS uses $scheme instead and I am going with that.

proxy_redirect also didn't work for me but that might be on my rather big config.

  • read the CONTRIBUTING guidelines
  • raised a GitHub issue or discussed it on the projects chat beforehand
  • added unit tests
  • added integration tests
  • updated documentation if needed
  • updated CHANGELOG.md

🔄 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/juanfont/headscale/pull/1698 **Author:** [@SuperSandro2000](https://github.com/SuperSandro2000) **Created:** 1/24/2024 **Status:** ❌ Closed **Base:** `main` ← **Head:** `patch-1` --- ### 📝 Commits (1) - [`1f0a511`](https://github.com/juanfont/headscale/commit/1f0a5117be5c6571fd7f581fe7088bf113345beb) docs/reverse-proxy: use standard map, correct X-Forwarded-Proto variable for nginx ### 📊 Changes **1 file changed** (+2 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `docs/reverse-proxy.md` (+2 -3) </details> ### 📄 Description …ble for nginx I tested this on NixOS. After switching from postgres back to sqlite and with the standard websocket proxy config, everything started to work. Initially the none standard map which differs from upstreams docs https://nginx.org/en/docs/http/websocket.html got me very confused and I thought this would be somehow required. Also $http_x_forwarded_proto doesn't exist, NixOS uses $scheme instead and I am going with that. proxy_redirect also didn't work for me but that might be on my rather big config. <!-- Please tick if the following things apply. You… --> - [x] read the [CONTRIBUTING guidelines](README.md#contributing) - [ ] raised a GitHub issue or discussed it on the projects chat beforehand - [ ] added unit tests - [ ] added integration tests - [x] updated documentation if needed - [ ] updated CHANGELOG.md <!-- If applicable, please reference the issue using `Fixes #XXX` and add tests to cover your new code. --> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
adam added the pull-request label 2025-12-29 03:20:27 +01:00
adam closed this issue 2025-12-29 03:20:27 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/headscale#2261