fix: do not throw when non buffer/string passed to isEncoded

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

Author: Alan Shaw

src/index.js

@@ -99,6 +99,11 @@ function isEncoded (bufOrString) {
     bufOrString = bufOrString.toString()
   }
 
+  // Ensure bufOrString is a string
+  if (Object.prototype.toString.call(bufOrString) !== '[object String]') {
+    return false
+  }
+
   const code = bufOrString.substring(0, 1)
   try {
     const base = getBase(code)

test/multibase.spec.js

@@ -230,3 +230,24 @@ describe('multibase.codes', () => {
     expect(Object.isFrozen(multibase.codes)).to.be.true()
   })
 })
+
+describe('multibase.isEncoded', () => {
+  it('should not throw for non string/buffer input', () => {
+    const invalidInputs = [
+      null,
+      undefined,
+      false,
+      0,
+      {},
+      [],
+      /[a-z]/,
+      () => {},
+      Symbol('test')
+    ]
+
+    invalidInputs.forEach(input => {
+      expect(() => multibase.isEncoded(input)).to.not.throw()
+      expect(multibase.isEncoded(input)).to.be.false()
+    })
+  })
+})