Merge pull request #55 from Remitly/preserve-host-config

sso-proxy: add preserve host configuration option
buzzfeed · Dec 1, 2018 · 6a6f920 · 6a6f920
2 parents c0ce81d + 9f81a78
commit 6a6f920
Show file tree

Hide file tree

Showing 4 changed files with 285 additions and 145 deletions.
diff --git a/internal/proxy/oauthproxy.go b/internal/proxy/oauthproxy.go
@@ -185,7 +185,9 @@ func NewReverseProxy(to *url.URL, config *UpstreamConfig) *httputil.ReverseProxy
 	proxy.Director = func(req *http.Request) {
 		req.Header.Add("X-Forwarded-Host", req.Host)
 		director(req)
-		req.Host = to.Host
+		if !config.PreserveHost {
+			req.Host = to.Host
+		}
 	}
 	return proxy
 }
@@ -214,7 +216,9 @@ func NewRewriteReverseProxy(route *RewriteRoute, config *UpstreamConfig) *httput
 
 		req.Header.Add("X-Forwarded-Host", req.Host)
 		director(req)
-		req.Host = target.Host
+		if !config.PreserveHost {
+			req.Host = target.Host
+		}
 	}
 	return proxy
 }