Retry with GET request in some situations when HEAD request fails

Munter · May 13, 2017 · bbe9aa0 · bbe9aa0
1 parent a9f96f7
commit bbe9aa0
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/lib/index.js b/lib/index.js
@@ -149,10 +149,10 @@ module.exports = function (options) {
         }
     }
 
-    function httpStatus(url, relations, shouldUseGetMethod) {
+    function httpStatus(url, relations, retryWithGetMethod) {
         return function (callback) {
             request({
-                method: shouldUseGetMethod ? 'get' : 'head',
+                method: retryWithGetMethod ? 'get' : 'head',
                 url: url.replace(/#.*$/, ''),
                 strictSSL: true,
                 gzip: true,
@@ -169,6 +169,11 @@ module.exports = function (options) {
                     var code = error.code;
 
                     if (code) {
+                        // Some servers send responses that request apparently handles badly when using the HEAD method...
+                        if (code === 'HPE_INVALID_CONSTANT' && !retryWithGetMethod) {
+                            return httpStatus(url, relations, true)(callback);
+                        }
+
                         if (code === 'ENOTFOUND') {
                             status = 'DNS Missing';
                         } else {
@@ -182,7 +187,7 @@ module.exports = function (options) {
 
                     callback(undefined, status);
                 } else {
-                    if (res.statusCode === 405) {
+                    if (!retryWithGetMethod && res.statusCode >= 400 && res.statusCode < 500) {
                         return httpStatus(url, relations, true)(callback);
                     }