Don't do dnslookups when API is down

- adds `.isUp` property to `api.js`
- also closes #54

Author: lidel

lib/api.js

@@ -7,8 +7,15 @@ const { Request } = require('sdk/request')
 const { XMLHttpRequest } = require('sdk/net/xhr')
 const { URL } = require('sdk/url')
 
+const pw = require('./peer-watch.js')
 const dnsCache = require('./dns-cache.js')
 
+Object.defineProperty(exports, 'isUp', {
+  get: function () {
+    return pw.peerCount != null
+  }
+})
+
 function apiUrl (path) {
   return URL('http://' + prefs.customGatewayHost + ':' + prefs.customApiPort + '/api/v0/' + path).toString()
 }
@@ -69,7 +76,7 @@ function isDnslinkPresent (fqdn) {
 
 function isDnslinkPresentCached (fqdn) {
   let ipfsSupport = dnsCache.get(fqdn)
-  if (typeof ipfsSupport === 'undefined') {
+  if (typeof ipfsSupport === 'undefined' && exports.isUp) {
     ipfsSupport = isDnslinkPresent(fqdn)
     if (ipfsSupport !== null) { // dont cache is API was down
       dnsCache.put(fqdn, ipfsSupport)

lib/gui.js

@@ -171,9 +171,9 @@ panel.port.on('open-preferences', hidePanelAnd(function () {
     onReady: function (tab) {
       tab.attach({
         contentScriptWhen: 'end',
-        contentScript: 'AddonManager.getAddonByID("' + self.id + '", function(aAddon) {' +
+        contentScript: 'if (typeof AddonManager !== \'undefined\'){AddonManager.getAddonByID("' + self.id + '", function(aAddon) {' +
           'unsafeWindow.gViewController.commands.cmd_showItemDetails.doCommand(aAddon, true);' +
-          '});'
+          '});}'
       })
     }
   })

lib/peer-watch.js

@@ -8,9 +8,11 @@ const gui = require('./gui.js')
 const gw = require('./gateways.js')
 const _ = require('sdk/l10n').get
 
+let peerCount = null
+
 function ipfsHealthCheck () {
   api.getSwarmPeers((peers) => {
-    const peerCount = (peers && peers.Strings) ? peers.Strings.length : null
+    peerCount = (peers && peers.Strings) ? peers.Strings.length : null
     gui.setButtonBadge(peerCount)
 
     if (prefs.automatic) {
@@ -38,3 +40,9 @@ require('sdk/system/unload').when(() => {
 })
 
 exports.triggerManualCheck = ipfsHealthCheck
+
+Object.defineProperty(exports, 'peerCount', {
+  get: function () {
+    return peerCount
+  }
+})

package.json

@@ -4,7 +4,7 @@
   "id": "[email protected]",
   "description": "Access IPFS resources via custom HTTP2IPFS gateway",
   "author": "Marcin Rataj",
-  "version": "1.5.0",
+  "version": "1.5.1",
   "license": "CC0-1.0",
   "homepage": "https://github.com/lidel/ipfs-firefox-addon",
   "icon": "resource://ipfs-firefox-addon-at-lidel-dot-org/data/icon-on-64.png",