Added true failsafe functionality

cw2018 · Nov 7, 2013 · 50a8aaa · 50a8aaa
1 parent 3873dd2
commit 50a8aaa
Showing 1 changed file with 11 additions and 5 deletions.
diff --git a/lib/helper.js b/lib/helper.js
@@ -174,9 +174,13 @@ function httpGetPromise(url, failsafe) {
   var cb = function (res) {
     var data = '';
 
-    if (res.statusCode != 200 && !failsafe) {
-      deferred.reject(new Error(
-          'The url (' + url + ') returned a status other than 200 "OK"'));
+    if (res.statusCode != 200) {
+      if (failsafe) {
+        deferred.resolve();
+      } else {
+        deferred.reject(new Error(
+          'The url (' + url + ') returned with status ' + res.statusCode));
+      }
     }
 
     res.on('data', function (chunk) {
@@ -195,9 +199,11 @@ function httpGetPromise(url, failsafe) {
   }
 
   get.on('error', function (err) {
-    if (!failsafe) {
+    if (failsafe) {
+      deferred.resolve();
+    } else {
       deferred.reject(new Error(
-          'The url (' + url + ') caused an error'));
+        'The url (' + url + ') caused an error'));
     }
   });