✨ add fetching grapher configs by UUID

functions/_common/grapherRenderer.ts

@@ -166,8 +166,20 @@ interface FetchGrapherConfigResult {
     etag: string | undefined
 }
 
+interface GrapherSlug {
+    type: "slug"
+    id: string
+}
+
+interface GrapherUuid {
+    type: "uuid"
+    id: string
+}
+
+type GrapherIdentifier = GrapherSlug | GrapherUuid
+
 export async function fetchUnparsedGrapherConfig(
-    slug: string,
+    identifier: GrapherIdentifier,
     env: Env,
     etag?: string
 ) {
@@ -178,10 +190,15 @@ export async function fetchUnparsedGrapherConfig(
         ? [env.GRAPHER_CONFIG_R2_BUCKET_PATH]
         : ["by-branch", env.CF_PAGES_BRANCH]
 
+    const directory =
+        identifier.type === "slug"
+            ? R2GrapherConfigDirectory.publishedGrapherBySlug
+            : R2GrapherConfigDirectory.byUUID
+
     const key = excludeUndefined([
         ...topLevelDirectory,
-        R2GrapherConfigDirectory.publishedGrapherBySlug,
-        `${slug}.json`,
+        directory,
+        `${identifier.id}.json`,
     ]).join("/")
 
     console.log("fetching grapher config from this key", key)
@@ -197,8 +214,8 @@ export async function fetchUnparsedGrapherConfig(
         const topLevelDirectory = env.GRAPHER_CONFIG_R2_BUCKET_FALLBACK_PATH
         const fallbackKey = excludeUndefined([
             topLevelDirectory,
-            R2GrapherConfigDirectory.publishedGrapherBySlug,
-            `${slug}.json`,
+            directory,
+            `${identifier.id}.json`,
         ]).join("/")
         fallbackUrl = new URL(
             fallbackKey,
@@ -211,11 +228,15 @@ export async function fetchUnparsedGrapherConfig(
 }
 
 export async function fetchGrapherConfig(
-    slug: string,
+    identifier: GrapherIdentifier,
     env: Env,
     etag?: string
 ): Promise<FetchGrapherConfigResult> {
-    const fetchResponse = await fetchUnparsedGrapherConfig(slug, env, etag)
+    const fetchResponse = await fetchUnparsedGrapherConfig(
+        identifier,
+        env,
+        etag
+    )
 
     if (fetchResponse.status === 404) {
         // we throw 404 errors instad of returning a 404 response so that the router
@@ -253,7 +274,10 @@ async function fetchAndRenderGrapherToSvg(
 ): Promise<string> {
     const grapherLogger = new TimeLogger("grapher")
 
-    const grapherConfigResponse = await fetchGrapherConfig(slug, env)
+    const grapherConfigResponse = await fetchGrapherConfig(
+        { type: "slug", id: slug },
+        env
+    )
 
     if (grapherConfigResponse.status === 404) {
         // we throw 404 errors instad of returning a 404 response so that the router

functions/grapher/[slug].ts

@@ -156,7 +156,11 @@ async function handleConfigRequest(
     const shouldCache = searchParams.get("nocache") === null
     console.log("Preparing json response for ", slug)
 
-    const grapherPageResp = await fetchUnparsedGrapherConfig(slug, env, etag)
+    const grapherPageResp = await fetchUnparsedGrapherConfig(
+        { type: "slug", id: slug },
+        env,
+        etag
+    )
 
     if (grapherPageResp.status === 304) {
         console.log("Returning 304 for ", slug)

functions/grapher/by-uuid/[uuid].ts

@@ -0,0 +1,69 @@
+import { Env } from "../thumbnail/[slug].js"
+import { fetchGrapherConfig } from "../../_common/grapherRenderer.js"
+import { IRequestStrict, Router, error, cors } from "itty-router"
+const { preflight, corsify } = cors({
+    allowMethods: "GET",
+})
+
+const router = Router<IRequestStrict, [URL, Env, string]>({
+    before: [preflight],
+    finally: [corsify],
+})
+router
+    .get(
+        "/grapher/by-uuid/:uuid.config.json",
+        async ({ params: { uuid } }, { searchParams }, env, etag) =>
+            handleConfigRequest(uuid, searchParams, env, etag)
+    )
+    .all("*", () => error(404, "Route not defined"))
+
+export const onRequestGet: PagesFunction = async (context) => {
+    // Makes it so that if there's an error, we will just deliver the original page before the HTML rewrite.
+    // Only caveat is that redirects will not be taken into account for some reason; but on the other hand the worker is so simple that it's unlikely to fail.
+    context.passThroughOnException()
+    const { request, env } = context
+    const url = new URL(request.url)
+
+    return router
+        .fetch(
+            request,
+            url,
+            { ...env, url },
+            request.headers.get("if-none-match")
+        )
+        .catch((e) => error(500, e))
+}
+
+async function handleConfigRequest(
+    uuid: string,
+    searchParams: URLSearchParams,
+    env: Env,
+    etag: string | undefined
+) {
+    const shouldCache = searchParams.get("nocache") === null
+    console.log("Preparing json response for uuid ", uuid)
+
+    const grapherPageResp = await fetchGrapherConfig(
+        { type: "uuid", id: uuid },
+        env,
+        etag
+    )
+
+    if (grapherPageResp.status === 304) {
+        return new Response(null, { status: 304 })
+    }
+
+    console.log("Grapher page response", grapherPageResp.grapherConfig.title)
+
+    const cacheControl = shouldCache
+        ? "public, s-maxage=3600, max-age=0, must-revalidate"
+        : "public, s-maxage=0, max-age=0, must-revalidate"
+
+    return new Response(JSON.stringify(grapherPageResp.grapherConfig), {
+        headers: {
+            "content-type": "application/json",
+            "Cache-Control": cacheControl,
+            ETag: grapherPageResp.etag,
+        },
+    })
+}