Add npmjs.com to weird hosts that add 'user-content-' in front of fra…

…gments
Munter · Nov 16, 2019 · 5aae7ea · 5aae7ea
1 parent 48cccd3
commit 5aae7ea
Showing 1 changed file with 8 additions and 5 deletions.
diff --git a/lib/index.js b/lib/index.js
@@ -12,6 +12,11 @@ const defaultSkipFilters = [require('./known-culprits/linkedin')];
 
 const hyperlinkUserAgent = `Hyperlink v${version} (https://www.npmjs.com/package/hyperlink)`;
 
+const userContentFragmentOrigins = [
+  'https://github.com',
+  'https://www.npmjs.com'
+];
+
 function checkCompatibility(asset, Class) {
   if (typeof Class === 'undefined') {
     Class = AssetGraph.Asset;
@@ -313,9 +318,7 @@ async function hyperlink(
                 } else {
                   reportTest({
                     ...redirectReport,
-                    expected: `${log[0].statusCode} ${url} --> 200 ${
-                      log[log.length - 1].redirectUri
-                    }`,
+                    expected: `${log[0].statusCode} ${url} --> 200 ${log[log.length - 1].redirectUri}`,
                     actual,
                     ok: false
                   });
@@ -752,9 +755,9 @@ async function hyperlink(
             actual: fragmentReport.expected
           });
         } else {
-          // Github does some weird things with mangling fragments and reversing it with runtime js
+          // Some hosts do weird things with mangling fragments and reversing it with runtime js
           if (
-            asset.origin === 'https://github.com' &&
+            userContentFragmentOrigins.includes(asset.origin) &&
             asset.ids.has(`user-content-${fragment.substr(1)}`)
           ) {
             reportTest({