From dcba6a095097aa776fc26499b58c75fb9ac6ecfe Mon Sep 17 00:00:00 2001
From: Karol Wypchlo <kwypchlo@gmail.com>
Date: Fri, 4 Mar 2022 18:48:36 +0100
Subject: [PATCH] fix hns domains using redirected skylink endpoint

---
 docker/nginx/conf.d/include/location-hns | 10 +++++++---
 docker/nginx/conf.d/server/server.api    |  1 -
 2 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/docker/nginx/conf.d/include/location-hns b/docker/nginx/conf.d/include/location-hns
index 62ff9729..fc1708e1 100644
--- a/docker/nginx/conf.d/include/location-hns
+++ b/docker/nginx/conf.d/include/location-hns
@@ -3,7 +3,8 @@ include /etc/nginx/conf.d/include/proxy-pass-internal;
 include /etc/nginx/conf.d/include/portal-access-check;
 
 # variable definititions - we need to define a variable to be able to access it in lua by ngx.var.something
-set $skylink ''; # placeholder for the raw 46 bit skylink
+set $skylink ''; # placeholder for the base64 skylink
+set $skylink_base32 ''; # placeholder for the base32 skylink
 
 # resolve handshake domain by requesting to /hnsres endpoint and assign correct values to $skylink and $rest
 rewrite_by_lua_block {
@@ -74,10 +75,13 @@ rewrite_by_lua_block {
     if ngx.var.path == "/" and skylink_rest ~= nil and skylink_rest ~= "" and skylink_rest ~= "/" then
         ngx.var.path = skylink_rest
     end
+
+    -- assign base32 skylink to be used in proxy_pass
+    ngx.var.skylink_base32 = require("skynet.skylink").base32(ngx.var.skylink)
 }
 
-# we proxy to another nginx location rather than directly to siad because we do not want to deal with caching here
-proxy_pass https://127.0.0.1/$skylink$path$is_args$args;
+proxy_set_header Host $skylink_base32.$skynet_portal_domain;
+proxy_pass $scheme://$server_addr$path$is_args$args;
 
 # in case siad returns location header, we need to replace the skylink with the domain name
 header_filter_by_lua_block {
diff --git a/docker/nginx/conf.d/server/server.api b/docker/nginx/conf.d/server/server.api
index 55bb0fe3..c24ecead 100644
--- a/docker/nginx/conf.d/server/server.api
+++ b/docker/nginx/conf.d/server/server.api
@@ -144,7 +144,6 @@ location /hns {
     set_by_lua_block $hns_domain { return string.match(ngx.var.uri, "/hns/([^/?]+)") }
     set_by_lua_block $path { return string.match(ngx.var.uri, "/hns/[^/?]+(.*)") }
 
-    proxy_set_header Host $host;
     include /etc/nginx/conf.d/include/location-hns;
 }