Try HTTP HEAD before falling back to HTTP GET for external assets. Al…

…so send some accept headers to improve odds of webserver responding. Refs #117
Munter · May 13, 2017 · 3e26dfa · 3e26dfa
1 parent 631eaba
commit 3e26dfa
Showing 1 changed file with 9 additions and 2 deletions.
diff --git a/lib/index.js b/lib/index.js
@@ -148,14 +148,17 @@ module.exports = function (options) {
         }
     }
 
-    function httpStatus(url, relations) {
+    function httpStatus(url, relations, shouldUseGetMethod) {
         return function (callback) {
             request({
+                method: shouldUseGetMethod ? 'get' : 'head',
                 url: url.replace(/#.*$/, ''),
                 strictSSL: true,
                 gzip: true,
                 headers: {
-                    'User-Agent': 'Hyperlink v' + version + ' (https://www.npmjs.com/package/hyperlink)'
+                    'User-Agent': 'Hyperlink v' + version + ' (https://www.npmjs.com/package/hyperlink)',
+                    'Accept': 'text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,*/*;q=0.8',
+                    'Accept-Encoding': 'gzip, deflate, sdch, br'
                 }
             }, function (error, res) {
                 var status,
@@ -178,6 +181,10 @@ module.exports = function (options) {
 
                     callback(undefined, status);
                 } else {
+                    if (res.statusCode === 405) {
+                        return httpStatus(url, relations, true);
+                    }
+
                     status = res.statusCode;
                     redirects = res.request.redirects || [];
                     var firstRedirectStatus = redirects[0] && redirects[0].statusCode;