feat: don't throw on invalid b58 string in getPeerId

package.json

@@ -62,4 +62,4 @@
     "greenkeeper[bot] <greenkeeper[bot]@users.noreply.github.com>",
     "npm-to-cdn-bot (by Forbes Lindesay) <npmcdn-to-unpkg-bot@users.noreply.github.com>"
   ]
-}
+}

src/index.js

@@ -106,7 +106,7 @@ Multiaddr.prototype.inspect = function inspect () {
 Multiaddr.prototype.protos = function protos () {
   return map(this.protoCodes(), function (code) {
     return extend(protocols(code))
-  // copy to prevent users from modifying the internal objs.
+    // copy to prevent users from modifying the internal objs.
   })
 }
 
@@ -249,13 +249,17 @@ Multiaddr.prototype.decapsulate = function decapsulate (addr) {
  * }
  */
 Multiaddr.prototype.getPeerId = function getPeerId () {
-  let b58str = this.stringTuples().filter((tuple) => {
-    if (tuple[0] === protocols.names['ipfs'].code) {
-      return true
-    }
-  })[0][1]
+  let b58str = null
+  try {
+    b58str = this.stringTuples().filter((tuple) => {
+      if (tuple[0] === protocols.names['ipfs'].code) {
+        return true
+      }
+    })[0][1]
+
+    bs58.decode(b58str)
+  } catch (e) {}
 
-  bs58.decode(b58str)
   return b58str
 }
 

test/index.spec.js

@@ -535,6 +535,22 @@ describe('helpers', () => {
     })
   })
 
+  describe('.getPeerId should parse id from multiaddr', () => {
+    it('parses extracts the peer Id from a multiaddr', () => {
+      expect(
+        multiaddr('/p2p-circuit/ipfs/QmcgpsyWgH8Y8ajJz1Cu72KnS5uo2Aa2LpzU7kinSupNKC').getPeerId()
+      ).to.equal('QmcgpsyWgH8Y8ajJz1Cu72KnS5uo2Aa2LpzU7kinSupNKC')
+    })
+  })
+
+  describe('.getPeerId should return null on missing peer id in multiaddr', () => {
+    it('parses extracts the peer Id from a multiaddr', () => {
+      expect(
+        multiaddr('/ip4/0.0.0.0/tcp/1234/utp').getPeerId()
+      ).to.be.null()
+    })
+  })
+
   describe('multiaddr.isMultiaddr', () => {
     it('handles different inputs', () => {
       expect(multiaddr.isMultiaddr(multiaddr('/'))).to.be.eql(true)