Changeset b5924eb in Klonkt for src/routes/guardian.js

Timestamp:

07/24/2026 08:07:47 PM (7 weeks ago)

Author:

Robin Genis <roboburr@…>

Branches:

main

Children:

c502242

Parents:

780a7c6

Message:

Guardian-PWA: cache-bust op de assets (stale JS was de "fire-and-forget")

Root van de klachten (geen verzend-bericht, lijkt fire-and-forget, en aan de
ward-kant "komt niet aan"): /assets wordt met een 1-jaar-cache geserveerd, dus
de browser bleef de oude guardian.js draaien na een deploy. Die oude versie
las S.pendingOffers (nu S.offers), dus de "Verzonden aanvragen"-sectie bleef
leeg en er kwam geen feedback. De server-kant werkte al: een echte offer
sound-fabrics -> beta wordt bezorgd en door beta opgeslagen (geverifieerd).

Fix: de guardian.js/css krijgen een ?v=<mtime>-versie die per deploy bumpt
(proces-restart herberekent de mtime), zodat een update meteen doorkomt.

Changed files:
src/routes/guardian.js

• ASSET_V uit de bestand-mtime; meegegeven aan de render

src/views/pages/guardian.ejs

• ?v= op guardian.css en guardian.js

remarks: bestaande browsers met de oude JS gecached: eenmalig hard-refreshen
(of de PWA herinstalleren). Daarna vangt de versie-query het af. De rest van
/assets houdt de 1-jaar-cache; alleen deze PWA-in-ontwikkeling wordt gebust.

-robo
Co-Authored-By: Claude Opus 4.8 <noreply@…>

File:

• src/routes/guardian.js (modified) (3 diffs)

src/routes/guardian.js

@@ -10 +10 @@
  */
 import express from 'express';
+import fs from 'fs';
+import path from 'path';
+import { fileURLToPath } from 'url';
 import db from '../config/database.js';
 import { requireAuth } from '../middleware/auth.js';
@@ -17 +20 @@
 
 const router = express.Router();
+
+// Cache-bust the PWA assets: /assets is served with a 1-year cache, so without
+// a per-build version the browser keeps running an old guardian.js/css after a
+// deploy (that was exactly the "no feedback / not arriving" bug). The version
+// is the file's mtime at process start; a redeploy restarts us and bumps it.
+const __dir = path.dirname(fileURLToPath(import.meta.url));
+function assetVersion(rel) {
+  try { return Math.floor(fs.statSync(path.join(__dir, '..', 'assets', rel)).mtimeMs).toString(36); }
+  catch { return '0'; }
+}
+const ASSET_V = { js: assetVersion('js/guardian.js'), css: assetVersion('css/guardian.css') };
 
 /** The acting site: ?site=slug when owned, else the user's first site. */
@@ -64 +78 @@
     sites,
     lang: L,
+    assetV: ASSET_V,
     t: (k, v) => i18nT(L, k, v),
     cspNonce: res.locals.cspNonce,