fix: base encode CIDs before logging or emitting them

BREAKING CHANGE: Emitted events have different bytes

The emitted events contain the stringified version of the CID, as we
change it to the base encoding the CID has, those bytes may be different
to previous versions of this module.

Though this shouldn't have any impact on any other modules as the
events are only used internally.

Author: achingbrain

src/notifications.js

@@ -34,7 +34,7 @@ class Notifications extends EventEmitter {
    * @return {void}
    */
   hasBlock (block) {
-    const str = `block:${block.cid.buffer.toString()}`
+    const str = `block:${block.cid}`
     this._log(str)
     this.emit(str, block)
   }
@@ -49,7 +49,7 @@ class Notifications extends EventEmitter {
    * @returns {void}
    */
   wantBlock (cid, onBlock, onUnwant) {
-    const cidStr = cid.buffer.toString()
+    const cidStr = cid.toString()
     this._log(`wantBlock:${cidStr}`)
 
     this._unwantListeners[cidStr] = () => {
@@ -80,7 +80,7 @@ class Notifications extends EventEmitter {
    * @returns {void}
    */
   unwantBlock (cid) {
-    const str = `unwant:${cid.buffer.toString()}`
+    const str = `unwant:${cid}`
     this._log(str)
     this.emit(str)
   }

test/notifications.spec.js

@@ -34,7 +34,7 @@ describe('Notifications', () => {
   it('hasBlock', (done) => {
     const n = new Notifications(peerId)
     const b = blocks[0]
-    n.once(`block:${b.cid.buffer.toString()}`, (block) => {
+    n.once(`block:${b.cid}`, (block) => {
       expect(b).to.eql(block)
       done()
     })