fix: make offline error retain stack (#1056)

Author: pgte

src/core/components/bitswap.js

@@ -10,15 +10,15 @@ module.exports = function bitswap (self) {
   return {
     wantlist: () => {
       if (!self.isOnline()) {
-        throw OFFLINE_ERROR
+        throw new Error(OFFLINE_ERROR)
       }
 
       const list = self._bitswap.getWantlist()
       return formatWantlist(list)
     },
     stat: () => {
       if (!self.isOnline()) {
-        throw OFFLINE_ERROR
+        throw new Error(OFFLINE_ERROR)
       }
 
       const stats = self._bitswap.stat()
@@ -29,7 +29,7 @@ module.exports = function bitswap (self) {
     },
     unwant: (key) => {
       if (!self.isOnline()) {
-        throw OFFLINE_ERROR
+        throw new Error(OFFLINE_ERROR)
       }
 
       // TODO: implement when https://github.com/ipfs/js-ipfs-bitswap/pull/10 is merged

src/core/components/pubsub.js

@@ -9,7 +9,7 @@ module.exports = function pubsub (self) {
   return {
     subscribe: (topic, options, handler, callback) => {
       if (!self.isOnline()) {
-        throw OFFLINE_ERROR
+        throw new Error(OFFLINE_ERROR)
       }
 
       if (typeof options === 'function') {
@@ -44,7 +44,7 @@ module.exports = function pubsub (self) {
 
     publish: promisify((topic, data, callback) => {
       if (!self.isOnline()) {
-        return setImmediate(() => callback(OFFLINE_ERROR))
+        return setImmediate(() => callback(new Error(OFFLINE_ERROR)))
       }
 
       if (!Buffer.isBuffer(data)) {
@@ -57,7 +57,7 @@ module.exports = function pubsub (self) {
 
     ls: promisify((callback) => {
       if (!self.isOnline()) {
-        return setImmediate(() => callback(OFFLINE_ERROR))
+        return setImmediate(() => callback(new Error(OFFLINE_ERROR)))
       }
 
       const subscriptions = Array.from(
@@ -69,7 +69,7 @@ module.exports = function pubsub (self) {
 
     peers: promisify((topic, callback) => {
       if (!self.isOnline()) {
-        return setImmediate(() => callback(OFFLINE_ERROR))
+        return setImmediate(() => callback(new Error(OFFLINE_ERROR)))
       }
 
       const peers = Array.from(self._pubsub.peers.values())

src/core/components/swarm.js

@@ -15,7 +15,7 @@ module.exports = function swarm (self) {
       }
 
       if (!self.isOnline()) {
-        return callback(OFFLINE_ERROR)
+        return callback(new Error(OFFLINE_ERROR))
       }
 
       const verbose = opts.v || opts.verbose
@@ -46,7 +46,7 @@ module.exports = function swarm (self) {
     // all the addrs we know
     addrs: promisify((callback) => {
       if (!self.isOnline()) {
-        return callback(OFFLINE_ERROR)
+        return callback(new Error(OFFLINE_ERROR))
       }
 
       const peers = values(self._peerInfoBook.getAll())
@@ -56,15 +56,15 @@ module.exports = function swarm (self) {
 
     localAddrs: promisify((callback) => {
       if (!self.isOnline()) {
-        return callback(OFFLINE_ERROR)
+        return callback(new Error(OFFLINE_ERROR))
       }
 
       callback(null, self._libp2pNode.peerInfo.multiaddrs.toArray())
     }),
 
     connect: promisify((maddr, callback) => {
       if (!self.isOnline()) {
-        return callback(OFFLINE_ERROR)
+        return callback(new Error(OFFLINE_ERROR))
       }
 
       if (typeof maddr === 'string') {
@@ -76,7 +76,7 @@ module.exports = function swarm (self) {
 
     disconnect: promisify((maddr, callback) => {
       if (!self.isOnline()) {
-        return callback(OFFLINE_ERROR)
+        return callback(new Error(OFFLINE_ERROR))
       }
 
       if (typeof maddr === 'string') {

src/core/utils.js

@@ -1,3 +1,3 @@
 'use strict'
 
-exports.OFFLINE_ERROR = new Error('This command must be run in online mode. Try running \'ipfs daemon\' first.')
+exports.OFFLINE_ERROR = 'This command must be run in online mode. Try running \'ipfs daemon\' first.'