chore: peer-discovery not using peer-info

BREAKING CHANGE: peer event emits an object with id and multiaddr instead of a peer-info

Author: vasco-santos

package.json

@@ -36,8 +36,7 @@
     "debug": "^4.1.1",
     "mafmt": "^7.0.0",
     "multiaddr": "^7.2.1",
-    "peer-id": "^0.13.5",
-    "peer-info": "^0.17.0"
+    "peer-id": "^0.13.5"
   },
   "pre-push": [
     "lint",

src/index.js

@@ -1,7 +1,6 @@
 'use strict'
 
 const PeerId = require('peer-id')
-const PeerInfo = require('peer-info')
 const multiaddr = require('multiaddr')
 const mafmt = require('mafmt')
 const { EventEmitter } = require('events')
@@ -50,7 +49,7 @@ class Bootstrap extends EventEmitter {
    * Emit each address in the list as a PeerInfo.
    */
   _discoverBootstrapPeers () {
-    this._list.forEach(async (candidate) => {
+    this._list.forEach((candidate) => {
       if (!mafmt.P2P.matches(candidate)) {
         return log.error('Invalid multiaddr')
       }
@@ -60,9 +59,10 @@ class Bootstrap extends EventEmitter {
       const peerId = PeerId.createFromB58String(ma.getPeerId())
 
       try {
-        const peerInfo = await PeerInfo.create(peerId)
-        peerInfo.multiaddrs.add(ma)
-        this.emit('peer', peerInfo)
+        this.emit('peer', {
+          id: peerId,
+          multiaddrs: [ma]
+        })
       } catch (err) {
         log.error('Invalid bootstrap peer id', err)
       }

test/bootstrap.spec.js

@@ -5,10 +5,12 @@ const chai = require('chai')
 chai.use(require('dirty-chai'))
 const { expect } = chai
 
+const mafmt = require('mafmt')
+const PeerId = require('peer-id')
+
 const Bootstrap = require('../src')
 const peerList = require('./default-peers')
 const partialValidPeerList = require('./some-invalid-peers')
-const mafmt = require('mafmt')
 
 describe('bootstrap', () => {
   it('should throw if no peer list is provided', () => {
@@ -42,10 +44,11 @@ describe('bootstrap', () => {
     })
 
     const p = new Promise((resolve) => {
-      r.once('peer', (peer) => {
-        const peerList = peer.multiaddrs.toArray()
-        expect(peerList.length).to.eq(1)
-        expect(mafmt.IPFS.matches(peerList[0].toString())).equals(true)
+      r.once('peer', ({ id, multiaddrs }) => {
+        expect(id).to.exist()
+        expect(PeerId.isPeerId(id)).to.eql(true)
+        expect(multiaddrs.length).to.eq(1)
+        expect(mafmt.IPFS.matches(multiaddrs[0].toString())).equals(true)
         resolve()
       })
     })