fix(Twitter): Add support for www subdomain

MichaelDeBoey · Oct 10, 2019 · 682a0ef · 682a0ef
1 parent 7189e93
commit 682a0ef
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/src/Twitter.js b/src/Twitter.js
@@ -4,7 +4,10 @@ import fetch from 'node-fetch';
 export const shouldTransform = url => {
   const { host, pathname } = new URL(url);
 
-  return host === 'twitter.com' && pathname.includes('/status/');
+  return (
+    (host === 'twitter.com' || host === 'www.twitter.com') &&
+    pathname.includes('/status/')
+  );
 };
 
 export const getHTML = url =>

diff --git a/src/__tests__/Twitter.js b/src/__tests__/Twitter.js
@@ -44,6 +44,10 @@ cases(
       url: 'https://twitter.com/foobar/status/123',
       valid: true,
     },
+    "status url having 'www' subdomain": {
+      url: 'https://www.twitter.com/foobar/status/123',
+      valid: true,
+    },
   }
 );