task: [isExternalLinkImplied] Use explicit list of "internal" hosts

cfpb · Dec 30, 2024 · e6b54e0 · e6b54e0
1 parent 76e221f
commit e6b54e0
Showing 1 changed file with 9 additions and 5 deletions.
diff --git a/src/components/Link.utils.tsx b/src/components/Link.utils.tsx
@@ -27,12 +27,16 @@ export const isExternalLinkImplied = (targetUrl: string): boolean => {
   const internalProtocols = ['mailto:'];
   if (internalProtocols.includes(parsed.protocol)) return false;
 
-  // Any subdomain of consumerfinance.gov or the current host
-  const isInternalDomain =
-    parsed.host.endsWith('consumerfinance.gov') ||
-    parsed.host === window.location.host;
+  // [Internal] Any subdomain of consumerfinance.gov or the current host
+  const internalHosts = [
+    'www.consumerfinance.gov',
+    'sblhelp.consumerfinance.gov',
+    window.location.host,
+  ];
 
-  return !isInternalDomain;
+  const isExternal = !internalHosts.includes(parsed.host);
+
+  return isExternal;
 };
 
 // External link icon w/ spacing