Merge pull request #12 from cagov/multi-hosts-per-domain

Fix host definition matching
cagov · Nov 20, 2023 · 06f55a5 · 06f55a5
2 parents 5318a70 + fe57508
commit 06f55a5
Showing 1 changed file with 1 addition and 12 deletions.
diff --git a/src/shared/hosts.js b/src/shared/hosts.js
@@ -11,16 +11,5 @@ const url = require("./url");
  * A matching host definition, or undefined if none are found.
  */
 exports.matchHostDef = (host, hostDefs) => {
-  const pHostUrl = url.parse(host);
-
-  if (pHostUrl && hostDefs) {
-    return hostDefs.find((hostDef) =>
-      hostDef.urls.some((hostDefUrl) => {
-        const pHostDefUrl = url.parse(hostDefUrl);
-        return pHostDefUrl.hostname === pHostUrl.hostname;
-      })
-    );
-  } else {
-    return undefined;
-  }
+  return hostDefs?.find((hostDef) => url.findMatch(hostDef.urls, host));
 };