Fix: add ookla both host and url

SukkaW · Feb 8, 2025 · 339eecb · 339eecb
1 parent b391de6
commit 339eecb
Showing 1 changed file with 12 additions and 3 deletions.
diff --git a/Build/build-speedtest-domainset.ts b/Build/build-speedtest-domainset.ts
@@ -27,9 +27,18 @@ interface SpeedTestServer {
 const getSpeedtestHostsGroupsPromise = $$fetch('https://speedtest-net-servers.cdn.skk.moe/servers.json')
   .then(res => res.json() as Promise<SpeedTestServer[]>)
   .then((data) => data.reduce<string[]>((prev, cur) => {
-    const hn = tldts.getHostname(cur.host || cur.url, { detectIp: false, validateHostname: true });
-    if (hn) {
-      prev.push(hn);
+    let hn: string | null = null;
+    if (cur.host) {
+      hn = tldts.getHostname(cur.host, { detectIp: false, validateHostname: true });
+      if (hn) {
+        prev.push(hn);
+      }
+    }
+    if (cur.url) {
+      hn = tldts.getHostname(cur.url, { detectIp: false, validateHostname: true });
+      if (hn) {
+        prev.push(hn);
+      }
     }
     return prev;
   }, []));