From 7987d0943827cddaf366d711c6fa519a6a4a4961 Mon Sep 17 00:00:00 2001
From: Adam Harrison <harrisonadamw@gmail.com>
Date: Tue, 17 Nov 2015 16:28:31 +0000
Subject: [PATCH] Revert "Merge pull request #1580 from
 weaveworks/1579-launch-proxy-mount-h"

This reverts commit 262352f8bbf97189a61c1fe4f21d8146a8731ab4, reversing
changes made to 312691376a560a9a9c932f6e23e68c8a9de3e084 due to a bug
whereby `weave env` returns a unix domain socket with an incorrect path.

This change will be superceeded by #1694.
---
 ...g_test.sh => 690_proxy_autoconfig_test.sh} |  7 +----
 weave                                         | 29 ++++---------------
 2 files changed, 6 insertions(+), 30 deletions(-)
 rename test/{690_proxy_config_test.sh => 690_proxy_autoconfig_test.sh} (81%)

diff --git a/test/690_proxy_config_test.sh b/test/690_proxy_autoconfig_test.sh
similarity index 81%
rename from test/690_proxy_config_test.sh
rename to test/690_proxy_autoconfig_test.sh
index 6293a3f5..ce6e18b2 100755
--- a/test/690_proxy_config_test.sh
+++ b/test/690_proxy_autoconfig_test.sh
@@ -2,7 +2,7 @@
 
 . ./config.sh
 
-start_suite "Various launch-proxy configurations"
+start_suite "Boot the proxy should only listen on client's interface"
 
 # Booting it over unix socket listens on unix socket
 run_on $HOST1 COVERAGE=$COVERAGE sudo -E weave launch-proxy
@@ -22,11 +22,6 @@ assert_raises "run_on $HOST1 sudo docker -H unix:///var/run/weave/weave.sock ps"
 assert_raises "proxy docker_on $HOST1 ps"
 weave_on $HOST1 stop-proxy
 
-# Booting it with -H outside /var/run/weave, still works
-socket="$(mktemp -d)/weave.sock"
-weave_on $HOST1 launch-proxy -H unix://$socket
-assert_raises "run_on $HOST1 sudo docker -H unix:///$socket ps" 0
-weave_on $HOST1 stop-proxy
 
 # Booting it over tls errors
 assert_raises "DOCKER_CLIENT_ARGS='--tls' weave_on $HOST1 launch-proxy" 1
diff --git a/weave b/weave
index 99684869..3be503b8 100755
--- a/weave
+++ b/weave
@@ -1266,36 +1266,17 @@ tls_arg() {
     PROXY_ARGS="$PROXY_ARGS $1 /home/weave/tls/$3.pem"
 }
 
-# TODO: Handle relative paths for args
-# TODO: Handle args with spaces
-host_arg() {
-  PROXY_HOST="$1"
-  if [ "$PROXY_HOST" != "${PROXY_HOST#unix://}" ]; then
-    host=$(dirname ${PROXY_HOST#unix://})
-    if [ "$host" = "${host#/}" ]; then
-      echo "When launching the proxy, unix sockets must be specified as an absolute path." >&2
-      exit 1
-    fi
-    if [ "$host" = "/var/run" ]; then
-      echo "WARNING: Proxying sockets whose immediate parent directory is /var/run is unreliable." >&2
-      echo "To work around this, put the unix socket in a subdirectory of /var/run, and symlink it into place." >&2
-    fi
-    PROXY_VOLUMES="$PROXY_VOLUMES -v $host:/host$host"
-    PROXY_ARGS="$PROXY_ARGS -H unix:///host$host/$(basename ${PROXY_HOST#unix://})"
-  else
-    PROXY_ARGS="$PROXY_ARGS -H $1"
-  fi
-}
-
 proxy_parse_args() {
     while [ $# -gt 0 ]; do
         case "$1" in
           -H)
-            host_arg "$2"
+            PROXY_HOST="$2"
+            PROXY_ARGS="$PROXY_ARGS $1 $2"
             shift
             ;;
           -H=*)
-            host_arg "${1#*=}"
+            PROXY_HOST="${1#*=}"
+            PROXY_ARGS="$PROXY_ARGS $1"
             ;;
           -no-detect-tls|--no-detect-tls)
             PROXY_TLS_DETECTION_DISABLED=1
@@ -1356,7 +1337,7 @@ proxy_args() {
           PROXY_HOST="tcp://0.0.0.0:12375"
           ;;
       esac
-      host_arg "$PROXY_HOST"
+      PROXY_ARGS="$PROXY_ARGS -H $PROXY_HOST"
     fi
 }