From f6e697d2ace2ea659c1b3f7e3dcef7493baa481c Mon Sep 17 00:00:00 2001
From: Brian Goff <cpuguy83@gmail.com>
Date: Thu, 12 Feb 2015 11:38:52 -0500
Subject: [PATCH] Do not use HTTP_PROXY for unix sock

Makes sure the CLI does not use HTTP_PROXY when connecting to unix
socket.
Also adds some tests to make sure this functionality works as expected.

Signed-off-by: Brian Goff <cpuguy83@gmail.com>
---
 api/client/cli.go                        |  2 +-
 integration-cli/docker_cli_proxy_test.go | 61 ++++++++++++++++++++++++
 2 files changed, 62 insertions(+), 1 deletion(-)
 create mode 100644 integration-cli/docker_cli_proxy_test.go

diff --git a/api/client/cli.go b/api/client/cli.go
index fbf204f32c..4c5eb2d0d6 100644
--- a/api/client/cli.go
+++ b/api/client/cli.go
@@ -155,7 +155,6 @@ func NewDockerCli(in io.ReadCloser, out, err io.Writer, keyFile string, proto, a
 
 	// The transport is created here for reuse during the client session
 	tr := &http.Transport{
-		Proxy:           http.ProxyFromEnvironment,
 		TLSClientConfig: tlsConfig,
 	}
 
@@ -168,6 +167,7 @@ func NewDockerCli(in io.ReadCloser, out, err io.Writer, keyFile string, proto, a
 			return net.DialTimeout(proto, addr, timeout)
 		}
 	} else {
+		tr.Proxy = http.ProxyFromEnvironment
 		tr.Dial = (&net.Dialer{Timeout: timeout}).Dial
 	}
 
diff --git a/integration-cli/docker_cli_proxy_test.go b/integration-cli/docker_cli_proxy_test.go
new file mode 100644
index 0000000000..7a08aef9aa
--- /dev/null
+++ b/integration-cli/docker_cli_proxy_test.go
@@ -0,0 +1,61 @@
+package main
+
+import (
+	"net"
+	"os/exec"
+	"strings"
+	"testing"
+)
+
+func TestCliProxyDisableProxyUnixSock(t *testing.T) {
+	cmd := exec.Command(dockerBinary, "info")
+	cmd.Env = []string{"HTTP_PROXY=http://127.0.0.1:9999"}
+
+	if out, _, err := runCommandWithOutput(cmd); err != nil {
+		t.Fatal(err, out)
+	}
+
+	logDone("cli proxy - HTTP_PROXY is not used when connecting to unix sock")
+}
+
+// Can't use localhost here since go has a special case to not use proxy if connecting to localhost
+// See http://golang.org/pkg/net/http/#ProxyFromEnvironment
+func TestCliProxyProxyTCPSock(t *testing.T) {
+	// get the IP to use to connect since we can't use localhost
+	addrs, err := net.InterfaceAddrs()
+	if err != nil {
+		t.Fatal(err)
+	}
+	var ip string
+	for _, addr := range addrs {
+		sAddr := addr.String()
+		if !strings.Contains(sAddr, "127.0.0.1") {
+			addrArr := strings.Split(sAddr, "/")
+			ip = addrArr[0]
+			break
+		}
+	}
+
+	if ip == "" {
+		t.Fatal("could not find ip to connect to")
+	}
+
+	d := NewDaemon(t)
+	if err := d.Start("-H", "tcp://"+ip+":2375"); err != nil {
+		t.Fatal(err)
+	}
+
+	cmd := exec.Command(dockerBinary, "info")
+	cmd.Env = []string{"DOCKER_HOST=tcp://" + ip + ":2375", "HTTP_PROXY=127.0.0.1:9999"}
+	if out, _, err := runCommandWithOutput(cmd); err == nil {
+		t.Fatal(err, out)
+	}
+
+	// Test with no_proxy
+	cmd.Env = append(cmd.Env, "NO_PROXY="+ip)
+	if out, _, err := runCommandWithOutput(exec.Command(dockerBinary, "info")); err != nil {
+		t.Fatal(err, out)
+	}
+
+	logDone("cli proxy - HTTP_PROXY is used for TCP sock")
+}