Revert "Merge pull request #931 from pedronasser/master"

This reverts commit 9ea05919, reversing
changes made to 2125ae5f.

caddyhttp/proxy/proxy.go

@@ -30,8 +30,6 @@ type Upstream interface {
 	Select() *UpstreamHost
 	// Checks if subpath is not an ignored path
 	AllowedPath(string) bool
-	// Is Upstream in transparent mode?
-	Transparent() bool
 }
 
 // UpstreamHostDownFunc can be used to customize how Down behaves.
@@ -96,7 +94,6 @@ func (p Proxy) ServeHTTP(w http.ResponseWriter, r *http.Request) (int, error) {
 	start := time.Now()
 	for time.Now().Sub(start) < tryDuration {
 		host := upstream.Select()
-
 		if host == nil {
 			return http.StatusBadGateway, errUnreachable
 		}
@@ -128,9 +125,6 @@ func (p Proxy) ServeHTTP(w http.ResponseWriter, r *http.Request) (int, error) {
 			if v, ok := host.UpstreamHeaders["Host"]; ok {
 				outreq.Host = replacer.Replace(v[len(v)-1])
 			}
-			if upstream.Transparent() {
-				host.UpstreamHeaders.Set("Host", host.Name)
-			}
 			// modify headers for request that will be sent to the upstream host
 			upHeaders := createHeadersByRules(host.UpstreamHeaders, r.Header, replacer)
 			for k, v := range upHeaders {

caddyhttp/proxy/proxy_test.go

@@ -687,10 +687,6 @@ func (u *fakeUpstream) AllowedPath(requestPath string) bool {
 	return true
 }
 
-func (u *fakeUpstream) Transparent() bool {
-	return true
-}
-
 // newWebSocketTestProxy returns a test proxy that will
 // redirect to the specified backendAddr. The function
 // also sets up the rules/environment for testing WebSocket
@@ -733,10 +729,6 @@ func (u *fakeWsUpstream) AllowedPath(requestPath string) bool {
 	return true
 }
 
-func (u *fakeWsUpstream) Transparent() bool {
-	return true
-}
-
 // recorderHijacker is a ResponseRecorder that can
 // be hijacked.
 type recorderHijacker struct {

caddyhttp/proxy/upstream.go

@@ -26,7 +26,6 @@ type staticUpstream struct {
 	Hosts              HostPool
 	Policy             Policy
 	insecureSkipVerify bool
-	transparent        bool
 
 	FailTimeout time.Duration
 	MaxFails    int32
@@ -55,7 +54,6 @@ func NewStaticUpstreams(c caddyfile.Dispenser) ([]Upstream, error) {
 			FailTimeout:       10 * time.Second,
 			MaxFails:          1,
 			MaxConns:          0,
-			transparent:       false,
 		}
 
 		if !c.Args(&upstream.from) {
@@ -294,7 +292,7 @@ func parseBlock(c *caddyfile.Dispenser, u *staticUpstream) error {
 		}
 		u.downstreamHeaders.Add(header, value)
 	case "transparent":
-		u.transparent = true
+		u.upstreamHeaders.Add("Host", "{host}")
 		u.upstreamHeaders.Add("X-Real-IP", "{remote}")
 		u.upstreamHeaders.Add("X-Forwarded-For", "{remote}")
 		u.upstreamHeaders.Add("X-Forwarded-Proto", "{scheme}")
@@ -373,11 +371,6 @@ func (u *staticUpstream) Select() *UpstreamHost {
 	return u.Policy.Select(pool)
 }
 
-// Transparent returns true if upstream in transparent mode
-func (u *staticUpstream) Transparent() bool {
-	return u.transparent
-}
-
 func (u *staticUpstream) AllowedPath(requestPath string) bool {
 	for _, ignoredSubPath := range u.IgnoredSubPaths {
 		if httpserver.Path(path.Clean(requestPath)).Matches(path.Join(u.From(), ignoredSubPath)) {

caddyhttp/proxy/upstream_test.go

@@ -209,8 +209,8 @@ func TestParseBlock(t *testing.T) {
 		for _, upstream := range upstreams {
 			headers := upstream.Select().UpstreamHeaders
 
-			if !upstream.Transparent() {
-				t.Errorf("Test %d: Upstream should be in transparent mode", i+1)
+			if _, ok := headers["Host"]; !ok {
+				t.Errorf("Test %d: Could not find the Host header", i+1)
 			}
 
 			if _, ok := headers["X-Real-Ip"]; !ok {