fix: properly serialize CID instances

The CID version agnostic tests ipfs-inactive/interface-js-ipfs-core#413 identified some functions were not properly serializing CID instances. This PR adds `cleanCID` step to several functions and also updates the `cleanCID` function to not assume buffer CIDs are CIDv0 multihashes.

depends on ipfs-inactive/interface-js-ipfs-core#413

License: MIT
Signed-off-by: Alan Shaw <alan.shaw@protocol.ai>

Author: Alan Shaw

src/files-regular/ls-pull-stream.js

@@ -3,6 +3,7 @@
 const moduleConfig = require('../utils/module-config')
 const pull = require('pull-stream')
 const deferred = require('pull-defer')
+const cleanCID = require('../utils/clean-cid')
 
 module.exports = (arg) => {
   const send = moduleConfig(arg)
@@ -13,6 +14,12 @@ module.exports = (arg) => {
       opts = {}
     }
 
+    try {
+      args = cleanCID(args)
+    } catch (err) {
+      return callback(err)
+    }
+
     const p = deferred.source()
 
     send({ path: 'ls', args: args, qs: opts }, (err, results) => {

src/files-regular/ls-readable-stream.js

@@ -2,6 +2,7 @@
 
 const moduleConfig = require('../utils/module-config')
 const Stream = require('readable-stream')
+const cleanCID = require('../utils/clean-cid')
 
 module.exports = (arg) => {
   const send = moduleConfig(arg)
@@ -12,6 +13,12 @@ module.exports = (arg) => {
       opts = {}
     }
 
+    try {
+      args = cleanCID(args)
+    } catch (err) {
+      return callback(err)
+    }
+
     const pt = new Stream.PassThrough({ objectMode: true })
 
     send({ path: 'ls', args: args, qs: opts }, (err, results) => {

src/files-regular/ls.js

@@ -2,6 +2,7 @@
 
 const promisify = require('promisify-es6')
 const moduleConfig = require('../utils/module-config')
+const cleanCID = require('../utils/clean-cid')
 
 module.exports = (arg) => {
   const send = moduleConfig(arg)
@@ -11,6 +12,13 @@ module.exports = (arg) => {
       callback = opts
       opts = {}
     }
+
+    try {
+      args = cleanCID(args)
+    } catch (err) {
+      return callback(err)
+    }
+
     send({
       path: 'ls',
       args: args,

src/utils/clean-cid.js

@@ -1,18 +1,17 @@
 'use strict'
 
-const bs58 = require('bs58')
 const CID = require('cids')
 
 module.exports = function (cid) {
   if (Buffer.isBuffer(cid)) {
-    cid = bs58.encode(cid)
+    return new CID(cid).toString()
   }
   if (CID.isCID(cid)) {
-    cid = cid.toBaseEncodedString()
+    return cid.toString()
   }
   if (typeof cid !== 'string') {
     throw new Error('unexpected cid type: ' + typeof cid)
   }
-  CID.validateCID(new CID(cid.split('/')[0]))
+  new CID(cid.split('/')[0]) // eslint-disable-line
   return cid
 }