fix: do not use assert in async funcs (#88)

libp2p · Apr 16, 2017 · 2e326e1 · 2e326e1
1 parent e4e9761
commit 2e326e1
Showing 1 changed file with 18 additions and 6 deletions.
diff --git a/src/index.js b/src/index.js
@@ -82,38 +82,50 @@ class Node extends EventEmitter {
 
     this.peerRouting = {
       findPeer: (id, callback) => {
-        assert(this._dht, 'DHT is not available')
+        if (!this._dht) {
+          return callback(new Error('DHT is not available'))
+        }
 
         this._dht.findPeer(id, callback)
       }
     }
 
     this.contentRouting = {
       findProviders: (key, timeout, callback) => {
-        assert(this._dht, 'DHT is not available')
+        if (!this._dht) {
+          return callback(new Error('DHT is not available'))
+        }
 
         this._dht.findProviders(key, timeout, callback)
       },
       provide: (key, callback) => {
-        assert(this._dht, 'DHT is not available')
+        if (!this._dht) {
+          return callback(new Error('DHT is not available'))
+        }
 
         this._dht.provide(key, callback)
       }
     }
 
     this.dht = {
       put: (key, value, callback) => {
-        assert(this._dht, 'DHT is not available')
+        if (!this._dht) {
+          return callback(new Error('DHT is not available'))
+        }
 
         this._dht.put(key, value, callback)
       },
       get: (key, callback) => {
-        assert(this._dht, 'DHT is not available')
+        if (!this._dht) {
+          return callback(new Error('DHT is not available'))
+        }
 
         this._dht.get(key, callback)
       },
       getMany (key, nVals, callback) {
-        assert(this._dht, 'DHT is not available')
+        if (!this._dht) {
+          return callback(new Error('DHT is not available'))
+        }
 
         this._dht.getMany(key, nVals, callback)
       }