chore/remove-peer-info-from-api

BREAKING CHANGE: findPeer returns id and addrs properties instead of peer-info instance

Author: vasco-santos

README.md

@@ -17,9 +17,9 @@ const DelegatedPeerRouting = require('libp2p-delegated-peer-routing')
 const routing = new DelegatedPeerRouing()
 
 try {
-  const peerInfo = await routing.findPeer('peerid')
+  const { id, addresses } = await routing.findPeer('peerid')
 
-  console.log('found peer details', peerInfo)
+  console.log('found peer details', id, addresses)
 } catch (err) {
   console.error(err)
 }

package.json

@@ -29,8 +29,7 @@
     "debug": "^4.1.1",
     "ipfs-http-client": "^43.0.1",
     "p-queue": "^6.2.1",
-    "peer-id": "~0.13.5",
-    "peer-info": "^0.17.1"
+    "peer-id": "~0.13.5"
   },
   "contributors": [
     "Jacob Heun <[email protected]>",

src/index.js

@@ -1,7 +1,6 @@
 'use strict'
 
 const PeerId = require('peer-id')
-const PeerInfo = require('peer-info')
 const createFindPeer = require('ipfs-http-client/src/dht/find-peer')
 const { default: PQueue } = require('p-queue')
 const debug = require('debug')
@@ -36,7 +35,7 @@ class DelegatedPeerRouting {
    * @param {PeerID} id
    * @param {object} options
    * @param {number} options.timeout How long the query can take. Defaults to 30 seconds
-   * @returns {Promise<PeerInfo>}
+   * @returns {Promise<{ id: CID, addrs: Multiaddr[] }>}
    */
   async findPeer (id, options = {}) {
     if (PeerId.isPeerId(id)) {
@@ -54,9 +53,10 @@ class DelegatedPeerRouting {
           }
         })
 
-        const peerInfo = new PeerInfo(PeerId.createFromCID(id))
-        addrs.forEach(addr => peerInfo.multiaddrs.add(addr))
-        return peerInfo
+        return {
+          id,
+          addrs
+        }
       })
     } catch (err) {
       if (err.message.includes('not found')) {

test/index.spec.js

@@ -111,15 +111,17 @@ describe('DelegatedPeerRouting', function () {
   describe('findPeers', () => {
     it('should be able to find peers via the delegate with a peer id string', async () => {
       const opts = delegatedNode.apiAddr.toOptions()
+
       const router = new DelegatedPeerRouting({
         protocol: 'http',
         port: opts.port,
         host: opts.host
       })
 
-      const peer = await router.findPeer(peerIdToFind.id)
-      expect(peer).to.exist()
-      expect(peer.id.toB58String()).to.eql(peerIdToFind.id)
+      const { id, addrs } = await router.findPeer(peerIdToFind.id)
+      expect(id).to.exist()
+      expect(addrs).to.exist()
+      expect(id).to.eql(peerIdToFind.id)
     })
 
     it('should be able to find peers via the delegate with a peerid', async () => {
@@ -130,9 +132,11 @@ describe('DelegatedPeerRouting', function () {
         host: opts.host
       })
 
-      const peer = await router.findPeer(PeerID.createFromB58String(peerIdToFind.id))
-      expect(peer).to.exist()
-      expect(peer.id.toB58String()).to.eql(peerIdToFind.id)
+      const { id, addrs } = await router.findPeer(PeerID.createFromB58String(peerIdToFind.id))
+      expect(id).to.exist()
+      expect(addrs).to.exist()
+
+      expect(id).to.eql(peerIdToFind.id)
     })
 
     it('should be able to specify a timeout', async () => {
@@ -143,9 +147,11 @@ describe('DelegatedPeerRouting', function () {
         host: opts.host
       })
 
-      const peer = await router.findPeer(PeerID.createFromB58String(peerIdToFind.id), { timeout: 2000 })
-      expect(peer).to.exist()
-      expect(peer.id.toB58String()).to.eql(peerIdToFind.id)
+      const { id, addrs } = await router.findPeer(PeerID.createFromB58String(peerIdToFind.id), { timeout: 2000 })
+      expect(id).to.exist()
+      expect(addrs).to.exist()
+
+      expect(id).to.eql(peerIdToFind.id)
     })
 
     it('should not be able to find peers not on the network', async () => {