From 6112150d04f73bc840acfe0435b2f33a17f49fbc Mon Sep 17 00:00:00 2001
From: Steven Allen <steven@stebalien.com>
Date: Wed, 13 Jun 2018 17:46:14 -0700
Subject: [PATCH 1/2] fix dht sharness tests

We can't currently put binary values due to API limitations. This worked before
because the DHT wasn't checking values on local put.

License: MIT
Signed-off-by: Steven Allen <steven@stebalien.com>
---
 test/sharness/t0170-dht.sh | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/test/sharness/t0170-dht.sh b/test/sharness/t0170-dht.sh
index 24aad3c56..3a90649ea 100755
--- a/test/sharness/t0170-dht.sh
+++ b/test/sharness/t0170-dht.sh
@@ -4,8 +4,8 @@ test_description="Test dht command"
 
 . lib/test-lib.sh
 
-TEST_DHT_VALUE="CAASpgIwggEiMA0GCSqGSIb3DQEBAQUAA4IBDwAwggEKAoIBAQC8hSwYY1FXqjT5M36O/Q5fBeDhXE5ePvGAeN3MIibfChqQpgqBbXQi1gAp4TQypSTKl/AMy7hfzsKauieim7jHMgIYAB4pLoBQD1qGVn/n7CqzAR3gDg9umIGuAy15oT0uaqMDqSepfnyxEyPDqfDklgvmS/MAwfBHjH2IPcMIaFgZ6d6gVlhmwuH8WVQ/geumDqyKuU9Jy+SUozmxEu2Baylg4fuqxaxoqOiPFZeWKSCFAngFj3NPmLApE0Fy48/eEZ+t7iP6s/raupP4+Jk/AFNDJNos4VxUnLJpZ1g6W5vYkkt1kXbMTaqxFVryCdCW2UEOwEzjGPGkcIE4RJrHAgMBAAE="
-TEST_DHT_PATH="/pk/QmepgFW7BHEtU4pZJdxaNiv75mKLLRQnPi1KaaXmQN4V1a"
+TEST_DHT_VALUE="foobar"
+TEST_DHT_PATH="/pk/QmbWTwYGcmdyK9CYfNBcfs9nhZs17a6FQ4Y8oea278xx41"
 
 # start iptb + wait for peering
 NUM_NODES=5
@@ -29,7 +29,7 @@ test_expect_success 'findpeer' '
 
 # ipfs dht put <key> <value>
 test_expect_success 'put with good keys' '
-  echo "$TEST_DHT_VALUE" | b64decode | ipfsi 0 dht put "$TEST_DHT_PATH" | sort >putted &&
+  ipfsi 0 dht put "$TEST_DHT_PATH" "$TEST_DHT_VALUE" | sort >putted &&
   [ -s putted ] ||
   test_fsh cat putted
 '
@@ -70,9 +70,9 @@ test_expect_success 'findprovs' '
 ## turns out to be the closest to what a key hashes to.
 # TODO: flaky. tracked by https://github.com/ipfs/go-ipfs/issues/2620
 test_expect_success 'query' '
-  ipfsi 3 dht query banana >actual &&
-  ipfsi 3 dht query apple >>actual &&
-  ipfsi 3 dht query pear >>actual &&
+  ipfsi 3 dht query "$(echo banana | ipfsi 3 add -q)" >actual &&
+  ipfsi 3 dht query "$(echo apple | ipfsi 3 add -q)" >>actual &&
+  ipfsi 3 dht query "$(echo pear | ipfsi 3 add -q)"  >>actual &&
   PEERS=$(wc -l actual | cut -d '"'"' '"'"' -f 1) &&
   [ -s actual ] ||
   test_might_fail test_fsh cat actual

From 384e2c40b24d3acf87171d690ec87cea0109db5d Mon Sep 17 00:00:00 2001
From: Steven Allen <steven@stebalien.com>
Date: Wed, 13 Jun 2018 17:59:38 -0700
Subject: [PATCH 2/2] add a failing test case for calling dht put with a bad
 key

see #5113

License: MIT
Signed-off-by: Steven Allen <steven@stebalien.com>
---
 test/sharness/t0170-dht.sh | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/test/sharness/t0170-dht.sh b/test/sharness/t0170-dht.sh
index 3a90649ea..90c9ec477 100755
--- a/test/sharness/t0170-dht.sh
+++ b/test/sharness/t0170-dht.sh
@@ -41,7 +41,15 @@ test_expect_success 'get with good keys' '
   ipfsi 1 dht get "/ipns/$PEERID_2" | grep -aq "/ipfs/$HASH"
 '
 
-test_expect_failure 'put with bad keys (issue #4611)' '
+test_expect_failure 'put with bad keys fails (issue #5113)' '
+  ipfsi 0 dht put "foo" "bar" >putted
+  ipfsi 0 dht put "/pk/foo" "bar" >>putted
+  ipfsi 0 dht put "/ipns/foo" "bar" >>putted
+  [ ! -s putted ] ||
+  test_fsh cat putted
+'
+
+test_expect_failure 'put with bad keys returns error (issue #4611)' '
   ! ipfsi 0 dht put "foo" "bar" &&
   ! ipfsi 0 dht put "/pk/foo" "bar" &&
   ! ipfsi 0 dht put "/ipns/foo" "bar"