about summary refs log tree commit diff
diff options
context:
space:
mode:
authorLuke Granger-Brown <git@lukegb.com>2020-11-29T03·18+0000
committerlukegb <lukegb@tvl.fyi>2020-11-29T11·50+0000
commitb344ae8783329b646b06240f3146d82ae25801f7 (patch)
tree5c5c7b02a17cebea94ffa46a738732d5751e6df0
parent844c84a38a1392a83becb0d6982f14a7d5fec536 (diff)
fix(cl.tvl.fyi): Correct Gerrit shortlink redirects. r/1959
Before: http://cl.tvl.fyi/123 -> https://cl.tvl.fyi:80/c/depot/+/123/
After: http://cl.tvl.fyi/123 -> https://cl.tvl.fyi/c/depot/+/123/

I think Jetty changed it's behaviour, and Gerrit is now configuring it
incorrectly.

Fixes #88.

Change-Id: I9238c0922b9f627e06eb81fa99dc748dada8909a
Reviewed-on: https://cl.tvl.fyi/c/depot/+/2202
Tested-by: BuildkiteCI
Reviewed-by: glittershark <grfn@gws.fyi>
-rw-r--r--ops/nixos/www/cl.tvl.fyi.nix3
1 files changed, 2 insertions, 1 deletions
diff --git a/ops/nixos/www/cl.tvl.fyi.nix b/ops/nixos/www/cl.tvl.fyi.nix
index 9383f38403..bcaab85f02 100644
--- a/ops/nixos/www/cl.tvl.fyi.nix
+++ b/ops/nixos/www/cl.tvl.fyi.nix
@@ -15,7 +15,8 @@
         location / {
           proxy_pass http://localhost:4778;
           proxy_set_header  X-Forwarded-For $remote_addr;
-          proxy_set_header  Host $host;
+          # The :443 suffix is a workaround for https://b.tvl.fyi/issues/88.
+          proxy_set_header  Host $host:443;
         }
       '';
     };