fix(swarm): move isConnected filter from addrs to peers (#901)

ipfs · Aug 31, 2017 · 6667f8a · 6667f8a
1 parent 6270d90
commit 6667f8a
Showing 1 changed file with 4 additions and 7 deletions.
diff --git a/src/core/components/swarm.js b/src/core/components/swarm.js
@@ -23,16 +23,14 @@ module.exports = function swarm (self) {
       // TODO: return latency and streams when verbose is set
       // we currently don't have this information
 
-      const peers = self._peerInfoBook.getAll()
-      const keys = Object.keys(peers)
-
-      const peerList = flatMap(keys, (id) => {
-        const peer = peers[id]
+      const peers = values(self._peerInfoBook.getAll())
+        .filter((peer) => peer.isConnected())
 
+      const peerList = flatMap(peers, (peer) => {
         return peer.multiaddrs.toArray().map((addr) => {
           const res = {
             addr: addr,
-            peer: peers[id]
+            peer: peer
           }
 
           if (verbose) {
@@ -53,7 +51,6 @@ module.exports = function swarm (self) {
       }
 
       const peers = values(self._peerInfoBook.getAll())
-        .filter((peer) => peer.isConnected())
 
       callback(null, peers)
     }),