Changeset 2a76184 in Klonkt for src/routes/guardian.js

Timestamp:

07/28/2026 09:11:34 AM (6 weeks ago)

Author:

Robin Genis <roboburr@…>

Branches:

main

Children:

65abc85

Parents:

bdb78e4

Message:

Guardians kunnen linkvoorbeelden aan- en uitzetten per ward

De eerste echte gated feature met een knop erbij. De gate bestond al server-side
(sites.external_embeds, uit voor een ward), maar er was geen enkele manier om
hem te bewegen: een ward zag dus nooit een linkvoorbeeld, en daar was niets aan
te doen behalve in de database duiken. Dat is precies de verkeerde plek voor een
beslissing die volgens het ontwerp bij de guardians hoort.

In de Guardian-PWA staat nu per ward een knop die de huidige stand toont en
omzet. Het endpoint accepteert het alleen van een COMMITTED guardian van die
ward, en alleen voor een ward die wij hosten: de instelling van een ward
elders hoort op de server van die ward thuis, dus die tonen we als
niet-instelbaar in plaats van te doen alsof.

Changed files:
src/routes/guardian.js

• POST /guardian/wards/embeds, met controle op guardian-van-deze-ward en lokale ward
• wardEmbedSetting(): huidige stand mee in de wards-lijst (null = niet van ons)

src/assets/js/guardian.js

• knop per ward, toont de stand en zet hem om

src/services/i18n.js

• guardian.embeds_on / embeds_off in nl, en, de

remarks: 219 tests groen. Federeren van deze instelling naar een remote ward is
een volgende stap (hoort bij shaer-3kp).

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

File:

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

src/routes/guardian.js

@@ -39 +39 @@
   const keys = ['sent', 'sent_retry', 'sending', 'not_found', 'failed', 'network',
     'pending', 'active', 'retract', 'release', 'release_confirm', 'open', 'push_unavailable',
+    'embeds_on', 'embeds_off',
     'accept', 'reject', 'complete', 'awaiting_others', 'coguard'];
   const s = Object.fromEntries(keys.map((k) => [k, i18nT(L, `guardian.${k}`)]));
@@ -56 +57 @@
     site: site.slug,
     me,
-    wards: Guardianship.listWards(site.slug),               // committed wards
+    // Committed wards, each carrying the gated settings a guardian may change.
+    // `embeds` is null for a ward we do not host: that setting lives on the
+    // ward's own server, so we show it as not-adjustable rather than lying.
+    wards: Guardianship.listWards(site.slug).map((w) => ({ ...w, embeds: wardEmbedSetting(w.other_uri) })),
     offers: Guardianship.offersCollection(`${me}/queues/offers`, site.slug, me).orderedItems,
     help,
@@ -266 +270 @@
   Guardianship.removeRelation(site.slug, 'guardian', uri);
   res.json({ ok: true });
+});
+
+/**
+ * The external-embeds setting of a ward we host: true/false when a guardian has
+ * decided, null when it is still on auto (which means off for a ward) or when
+ * the ward lives elsewhere and the setting is not ours to show.
+ */
+function wardEmbedSetting(uri) {
+  const base = (process.env.PUBLIC_BASE_URL || '').replace(/\/+$/, '');
+  if (!base || !String(uri || '').startsWith(`${base}/`)) return null;
+  const slug = String(uri).trim().replace(/\/+$/, '').split('/').pop();
+  const row = slug ? db.prepare('SELECT external_embeds FROM sites WHERE slug = ?').get(slug) : null;
+  if (!row) return null;
+  return row.external_embeds === null || row.external_embeds === undefined ? false : row.external_embeds === 1;
+}
+
+// ── Gated feature: may this ward see external (non-fediverse) embeds? ──
+// The first real gated setting (FEP-633c §5-style). The gate itself is applied
+// server-side when the feed is serialised, so this endpoint is the only way it
+// can move, and only a committed guardian of THAT ward may move it.
+router.post('/wards/embeds', requireAuth, express.json({ limit: '4kb' }), (req, res) => {
+  const site = siteForUser(req);
+  if (!site) return res.status(404).json({ error: 'no_site' });
+  const uri = String(req.body?.uri || '').trim();
+  const allow = req.body?.allow === true;
+  if (!uri) return res.status(400).json({ error: 'empty_uri' });
+  // Only a guardian of this ward, and only for a ward we host: a setting on a
+  // remote ward belongs to that ward's own server (federating it is Fase 4).
+  const isMyWard = Guardianship.listWards(site.slug).some((w) => w.other_uri === uri);
+  if (!isMyWard) return res.status(403).json({ error: 'not_your_ward' });
+  const base = (process.env.PUBLIC_BASE_URL || '').replace(/\/+$/, '');
+  const slug = (base && uri.startsWith(`${base}/`)) ? uri.trim().replace(/\/+$/, '').split('/').pop() : null;
+  const ward = slug ? db.prepare('SELECT id, slug FROM sites WHERE slug = ?').get(slug) : null;
+  if (!ward) return res.status(400).json({ error: 'remote_ward_not_supported' });
+  db.prepare('UPDATE sites SET external_embeds = ? WHERE id = ?').run(allow ? 1 : 0, ward.id);
+  res.json({ ok: true, allow });
 });