canonicalizeurl refactoring to avoid source domain

Author: lcflight

astro.config.ts

@@ -3,6 +3,7 @@ import prefetch from "@astrojs/prefetch";
 import getRedirects from "./src/lib/getRedirects";
 
 import sitemap from "@astrojs/sitemap";
+import "dotenv/config";
 
 // https://astro.build/config
 export default defineConfig({

package.json

@@ -31,6 +31,7 @@
     "@fontsource-variable/lora": "^5.0.9",
     "@fontsource/source-sans-pro": "^5.0.8",
     "astro": "^3.1.0",
+    "dotenv": "^16.3.1",
     "feather-icons": "^4.29.1",
     "findup-sync": "^5.0.0",
     "gray-matter": "^4.0.3",

pnpm-lock.yaml

@@ -3,8 +3,26 @@ import canonicalizeUrl from "./canonicalizeUrl";
 
 describe("canonicalizeUrl", () => {
   it("adds schema if missing", async () => {
-    const result = canonicalizeUrl("padm.us/psychpricing");
+    const result = canonicalizeUrl("doc.bmndr.co/psychpricing");
 
     expect(result).toContain("https://");
   });
+
+  it("replaces domain", async () => {
+    const result = canonicalizeUrl("doc.bmndr.co/psychpricing");
+
+    expect(result).not.toContain("doc.bmndr.co");
+  });
+
+  it("uses source domain", async () => {
+    const result = canonicalizeUrl("doc.bmndr.co/psychpricing");
+
+    expect(result).not.toContain("undefined");
+  });
+
+  it("returns the correct url", async () => {
+    const result = canonicalizeUrl("doc.bmndr.co/psychpricing");
+
+    expect(result).toEqual("https://the_source_domain/psychpricing/export/txt");
+  });
 });

src/lib/canonicalizeUrl.spec.ts

@@ -1,4 +1,7 @@
+import env from "./env";
+
 export default function canonicalizeUrl(url: string): string {
+  const domain = env("SOURCE_DOMAIN");
   // Start with basic cleanup canonicalization applicable to all URLs...
   url = url.trim(); // trim whitespace
   if (!url.match(/^https?:\/\//i)) url = "https://" + url; // add protocol
@@ -12,14 +15,21 @@ export default function canonicalizeUrl(url: string): string {
     /^(https?:\/\/doc\.bmndr\.com)\/([^/]+)$/,
     /^(https?:\/\/doc\.beeminder\.com)\/([^/]+)$/,
   ];
+
   for (const regex of sugarlist) {
-    url = url.replace(regex, "https://" + "padm" + "." + "us" + "/" + "$2");
+    url = url.replace(regex, `https://${domain}/$2`);
   }
 
-  // Following is a special case for the raw padm.us URL just in case:
-  url = url.replace(/^(https?:\/\/padm\.us)$/, "$1/public/export/txt");
+  // Following is a special case for the raw source URL just in case:
+  url = url.replace(
+    new RegExp(`^(https?://${domain})$`),
+    "$1/public/export/txt",
+  );
   // And finally the key transformation: append /export/txt for etherpad:
-  url = url.replace(/^(https?:\/\/padm\.us)\/([^/]+)$/, "$1/$2/export/txt");
+  url = url.replace(
+    new RegExp(`^(https?://${domain})/([^/]+)$`),
+    "$1/$2/export/txt",
+  );
 
   return url;
 }

src/lib/canonicalizeUrl.ts

@@ -62,5 +62,10 @@ beforeEach(() => {
     timeEnd: vi.fn(),
   });
 
+  vi.stubGlobal("process", {
+    ...process,
+    env: { ...process.env, SOURCE_DOMAIN: "the_source_domain" },
+  });
+
   __reset();
 });