From db45c4d0d9ab0b8f9a4f2120f44126cb771edf6c Mon Sep 17 00:00:00 2001
From: Steven Allen <steven@stebalien.com>
Date: Sun, 29 Jul 2018 11:42:02 -0700
Subject: [PATCH] remove bitswap unwant

This command messes with internal state and doesn't even work at the moment. If
you don't want a block, you should cancel the request that's trying to fetch it.

fixes #5295

License: MIT
Signed-off-by: Steven Allen <steven@stebalien.com>
---
 core/commands/bitswap.go             | 48 ----------------------------
 core/commands/commands_test.go       |  1 -
 misc/completion/ipfs-completion.bash | 12 +------
 3 files changed, 1 insertion(+), 60 deletions(-)

diff --git a/core/commands/bitswap.go b/core/commands/bitswap.go
index f6686f595..89bb81152 100644
--- a/core/commands/bitswap.go
+++ b/core/commands/bitswap.go
@@ -16,7 +16,6 @@ import (
 	cmds "gx/ipfs/QmPTfgFTo9PFr1PvPKyKoeMgBvYPh6cX3aDP7DHKVbnCbi/go-ipfs-cmds"
 	peer "gx/ipfs/QmQsErDt8Qgw1XrsXf2BpEzDgGWtB1YLsTAARBup5b6B9W/go-libp2p-peer"
 	cmdkit "gx/ipfs/QmSP88ryZkHSRn1fnngAaV2Vcn63WUJzAavnRM9CVdU1Ky/go-ipfs-cmdkit"
-	cid "gx/ipfs/QmZFbDTY9jfSBms2MchvYM9oYRbAF19K7Pby47yDBfpPrb/go-cid"
 )
 
 var BitswapCmd = &cmds.Command{
@@ -28,58 +27,11 @@ var BitswapCmd = &cmds.Command{
 	Subcommands: map[string]*cmds.Command{
 		"stat":      bitswapStatCmd,
 		"wantlist":  lgc.NewCommand(showWantlistCmd),
-		"unwant":    lgc.NewCommand(unwantCmd),
 		"ledger":    lgc.NewCommand(ledgerCmd),
 		"reprovide": lgc.NewCommand(reprovideCmd),
 	},
 }
 
-var unwantCmd = &oldcmds.Command{
-	Helptext: cmdkit.HelpText{
-		Tagline: "Remove a given block from your wantlist.",
-	},
-	Arguments: []cmdkit.Argument{
-		cmdkit.StringArg("key", true, true, "Key(s) to remove from your wantlist.").EnableStdin(),
-	},
-	Run: func(req oldcmds.Request, res oldcmds.Response) {
-		nd, err := req.InvocContext().GetNode()
-		if err != nil {
-			res.SetError(err, cmdkit.ErrNormal)
-			return
-		}
-
-		if !nd.OnlineMode() {
-			res.SetError(ErrNotOnline, cmdkit.ErrClient)
-			return
-		}
-
-		bs, ok := nd.Exchange.(*bitswap.Bitswap)
-		if !ok {
-			res.SetError(e.TypeErr(bs, nd.Exchange), cmdkit.ErrNormal)
-			return
-		}
-
-		var ks []*cid.Cid
-		for _, arg := range req.Arguments() {
-			c, err := cid.Decode(arg)
-			if err != nil {
-				res.SetError(err, cmdkit.ErrNormal)
-				return
-			}
-
-			ks = append(ks, c)
-		}
-
-		// TODO: This should maybe find *all* sessions for this request and cancel them?
-		// (why): in reality, i think this command should be removed. Its
-		// messing with the internal state of bitswap. You should cancel wants
-		// by killing the command that caused the want.
-		bs.CancelWants(ks, 0)
-
-		res.SetOutput(nil)
-	},
-}
-
 var showWantlistCmd = &oldcmds.Command{
 	Helptext: cmdkit.HelpText{
 		Tagline: "Show blocks currently on the wantlist.",
diff --git a/core/commands/commands_test.go b/core/commands/commands_test.go
index c0abceaf2..a365f3e1b 100644
--- a/core/commands/commands_test.go
+++ b/core/commands/commands_test.go
@@ -73,7 +73,6 @@ func TestCommands(t *testing.T) {
 		"/bitswap/ledger",
 		"/bitswap/reprovide",
 		"/bitswap/stat",
-		"/bitswap/unwant",
 		"/bitswap/wantlist",
 		"/block",
 		"/block/get",
diff --git a/misc/completion/ipfs-completion.bash b/misc/completion/ipfs-completion.bash
index 27f83fdef..c1487cb34 100644
--- a/misc/completion/ipfs-completion.bash
+++ b/misc/completion/ipfs-completion.bash
@@ -27,7 +27,7 @@ _ipfs_add()
 
 _ipfs_bitswap()
 {
-    ipfs_comp "ledger stat unwant wantlist --help"
+    ipfs_comp "ledger stat wantlist --help"
 }
 
 _ipfs_bitswap_ledger()
@@ -40,21 +40,11 @@ _ipfs_bitswap_stat()
     _ipfs_help_only
 }
 
-_ipfs_bitswap_unwant()
-{
-    _ipfs_help_only
-}
-
 _ipfs_bitswap_wantlist()
 {
     ipfs_comp "--peer= --help"
 }
 
-_ipfs_bitswap_unwant()
-{
-    _ipfs_help_only
-}
-
 _ipfs_block()
 {
     _ipfs_comp "get put rm stat --help"