Close streams when done with them

[Stebalien]

Our stream muxers all support half-open connections. Therefore, if you receive an EOF on a stream, you must send an EOF on that stream to fully close it. Otherwise, the other end will (if correctly implemented) sit around waiting for the EOF.

I know that the part of bitswap that accepts blocks from peers isn't doing this and I wouldn't be surprised if this doesn't happen in other places as well (stream closing has been a persistent problem in go-ipfs as well).

This showed up in ipfs/interop#28.

[Stebalien]

This should probably be difficulty:easy. js-ipfs just needs to close the stream when done.

[daviddias]

@Stebalien I'm pretty sure we always close when done. Couldn't find any location where that is not happening.

[Stebalien]

You sure you don't need something like:

diff --git a/src/network.js b/src/network.js
index 0e914c5..ae91bfa 100644
--- a/src/network.js
+++ b/src/network.js
@@ -81,6 +81,7 @@ class Network {
         if (err) {
           this.bitswap._receiveError(err)
         }
+        pull(pull.empty(), conn)
       })
     )
   }

We have a duplex stream so we can't just read until the end, we have to tell the other side that we aren't sending anything (unless pull does this under the covers...).