Fix bug with assuming there is a response on the error object

Author: tylercollier

lib/sync/downloader.js

@@ -133,6 +133,7 @@ module.exports = function(mlsSourceName, configBundle, eventEmitter, loggerArg)
       }
       auth = await ensureAuthIsNotExpired(auth)
       let loopCount = 0
+      const maxLoops = 3
       while (!metadataString) {
         loopCount++
         try {
@@ -156,7 +157,8 @@ module.exports = function(mlsSourceName, configBundle, eventEmitter, loggerArg)
             // })
             // .catch(catcher('get metadata'))
         } catch (error) {
-          if (loopCount <= 3 && (error.code === 'EAI_AGAIN' || !'debug' || error.response.status >= 500)) {
+          if (loopCount <= maxLoops
+            && (error.code === 'EAI_AGAIN' || !'debug' || (error.response && error.response.status >= 500))) {
             // Will loop
             console.log(`There was a problem with the request, trying again (${loopCount})`)
           } else {
@@ -322,7 +324,8 @@ module.exports = function(mlsSourceName, configBundle, eventEmitter, loggerArg)
           }
         })
         .catch(error => {
-          if (loopCount <= maxLoops && (error.code === 'EAI_AGAIN' || !'debug' || error.response.status >= 500)) {
+          if (loopCount <= maxLoops
+            && (error.code === 'EAI_AGAIN' || !'debug' || (error.response && error.response.status >= 500))) {
             // Will loop
             console.log(`There was a problem with the request, trying again (${loopCount})`)
           } else {