From 12ce7d4c18b5506998a8a034cfdb179322b530d2 Mon Sep 17 00:00:00 2001
From: Matthias Radestock <matthias.radestock@gmail.com>
Date: Mon, 4 May 2015 09:59:23 +0100
Subject: [PATCH 1/2] add some tests of `weave expose/hide` functionality

---
 test/130_expose_test.sh | 61 +++++++++++++++++++++++++++++++++++++++++
 1 file changed, 61 insertions(+)
 create mode 100755 test/130_expose_test.sh

diff --git a/test/130_expose_test.sh b/test/130_expose_test.sh
new file mode 100755
index 00000000..8a9917b4
--- /dev/null
+++ b/test/130_expose_test.sh
@@ -0,0 +1,61 @@
+#! /bin/bash
+
+. ./config.sh
+
+C1=c1:10.2.1.34
+C2=c2:10.2.1.37
+C3=c3:10.2.2.34
+C4=c4:10.2.2.37
+EXP=10.2.1.101
+
+PING="ping -nq -W 1 -c 1"
+
+weave_on1() {
+    assert_raises "weave_on $HOST1 $@"
+}
+
+run_on1() {
+    assert_raises "run_on   $HOST1 $@"
+}
+
+exec_on1() {
+    assert_raises "exec_on  $HOST1 $@"
+}
+
+check_container_connectivity() {
+    exec_on1 "${C1%:*} $PING ${C2#*:}"
+    exec_on1 "${C3%:*} $PING ${C4#*:}"
+    # fails due to #620
+    # exec_on1 "${C3#:} ! $PING ${C1#*:}"
+}
+
+start_suite "exposing weave network to host"
+
+weave_on $HOST1 launch
+
+for c in $C1 $C2 $C3 $C4; do
+    weave_on $HOST1 run ${c#*:}/24 -t --name=${c%:*} ubuntu
+done
+
+# absence of host connectivity by default
+run_on1 "! $PING ${C1#*:}"
+check_container_connectivity
+
+# host connectivity after 'expose'
+weave_on1 "expose $EXP/24"
+run_on1   "  $PING ${C1#*:}"
+run_on1   "! $PING ${C3#*:}"
+check_container_connectivity
+
+# idempotence of 'expose'
+weave_on1 "expose $EXP/24"
+run_on1   "$PING ${C1#*:}"
+
+# no host connectivity after 'hide'
+weave_on1 "hide $EXP/24"
+run_on1   "! $PING ${C1#*:}"
+
+# idempotence of 'hide'
+weave_on1 "hide $EXP/24"
+
+end_suite

From d8f1a044678f598066cd75ffa520b0bfdbff638a Mon Sep 17 00:00:00 2001
From: Matthias Radestock <matthias.radestock@gmail.com>
Date: Mon, 4 May 2015 09:59:58 +0100
Subject: [PATCH 2/2] only NAT when crossing between subnets otherwise we can
 end up NATing traffic inside a subnet, placing it outside the subnet and thus
 breaking connectivity.

---
 weave | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/weave b/weave
index 39b17fb7..786c62ac 100755
--- a/weave
+++ b/weave
@@ -817,8 +817,8 @@ case "$COMMAND" in
             then
                 ip addr add dev $BRIDGE $CIDR
                 arp_update $BRIDGE $CIDR
-                add_iptables_rule nat WEAVE -o $BRIDGE ! -s $CIDR -j MASQUERADE
-                add_iptables_rule nat WEAVE -s $CIDR ! -o $BRIDGE -j MASQUERADE
+                add_iptables_rule nat WEAVE -d $CIDR ! -s $CIDR -j MASQUERADE
+                add_iptables_rule nat WEAVE -s $CIDR ! -d $CIDR -j MASQUERADE
                 if [ "$FQDN" ]; then
                     http_call $DNS_CONTAINER_NAME $DNS_HTTP_PORT PUT /name/weave:expose/${CIDR%/*} --data-urlencode "fqdn=$FQDN" 2>/dev/null || true
                 fi
@@ -834,8 +834,8 @@ case "$COMMAND" in
             if ip addr show dev $BRIDGE | grep -qF $CIDR
             then
                 ip addr del dev $BRIDGE $CIDR
-                delete_iptables_rule nat WEAVE -o $BRIDGE ! -s $CIDR -j MASQUERADE
-                delete_iptables_rule nat WEAVE -s $CIDR ! -o $BRIDGE -j MASQUERADE
+                delete_iptables_rule nat WEAVE -d $CIDR ! -s $CIDR -j MASQUERADE
+                delete_iptables_rule nat WEAVE -s $CIDR ! -d $CIDR -j MASQUERADE
                 http_call $DNS_CONTAINER_NAME $DNS_HTTP_PORT DELETE /name/weave:expose/${CIDR%/*} 2>/dev/null || true
             fi
         done