Changeset e27b8db in Klonkt for test/gated-settings.test.js

Timestamp:

07/29/2026 10:04:36 AM (6 weeks ago)

Author:

Robin Genis <roboburr@…>

Branches:

main

Children:

329873e

Parents:

88d7c8f

Message:

Afspelen in de app als tweede gated feature, en het gat in de gate

Bij het uitzoeken van de YouTube-vraag bleek de gate lek. De web-Krant bouwt de
speler uit de inhoud van de post via timelineEmbedHtml, en dat pad raakte
gateEmbeds nooit. Een ward wiens guardians niets hadden toegestaan kreeg dus de
volledige YouTube-speler op het web, terwijl de app niets liet zien: het zware
ding open, het lichte dicht. Precies omgekeerd.

Nu zijn het twee besluiten, want het zijn twee dingen. Zien dat er een filmpje
is, is niet hetzelfde als het scherm afstaan aan de motor van een derde partij,
compleet met eindscherm en volgende-video. shaer:externalEmbeds houdt de kaart,
shaer:externalPlayback de speler, allebei standaard uit voor een ward, en
afspelen vereist de kaart: je kunt niet spelen wat je niet mag zien.

En het antwoord op Robins vraag over de links: die vallen er ook onder. De gate
verborg tot nu toe alleen het plaatje terwijl de kale link eronder gewoon
aantikbaar bleef, dus de deur stond open met een doek eroverheen. Staat de gate
dicht, dan toont de kaart zich nog wel maar is hij geen deur meer.

De server bepaalt wat gespeeld mag worden, niet de client: hij levert
shaer:playerUrl mee, alleen bij een open gate en alleen in de privacy-variant
(youtube-nocookie met rel=0, of de eigen speler van de PeerTube-instance). De
app houdt zo geen lijst van hosts bij; hij speelt wat hij krijgt aangereikt.

Changed files:
src/config/database.js

• kolom sites.external_playback

src/services/guardianship/notes.js

• externalPlaybackAllowed naast externalEmbedsAllowed

src/services/guardianship/gated.js

• shaer:externalPlayback in de feature-tabel

src/services/ActivityPubService.js

• timelineEmbed voegt shaer:playerUrl toe als afspelen mag; playerUrlFor kent alleen privacy-varianten en weigert de rest

src/routes/activitypub.js

• shaer:capabilities op de owner-only inbox-read: wat mag dit account
• de embed draagt de speler-URL alleen bij een open playback-gate

src/routes/posts.js

• het gat gedicht: de speler-iframe op de web-Krant valt nu onder de gate

src/routes/guardian.js

• de voorstel-route is feature-bewust; het lokale pad stuurt nu ook door

src/assets/js/guardian.js

• tweede knop in het paneel, alleen zichtbaar als de kaart al aan staat

src/services/i18n.js

• de labels in nl, en, de

test/gated-settings.test.js

• drie tests: de speler-URL rijdt alleen mee bij een open gate, een pagina die we niet framen blijft een thumbnail, en afspelen vereist de kaart

remarks: 280 tests groen. Niets geforceerd: beide gates staan standaard uit
voor een ward en twee van de drie guardians moeten nog steeds akkoord gaan.

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

File:

• test/gated-settings.test.js (modified) (1 diff)

test/gated-settings.test.js

@@ -134 +134 @@
     'two of three agreed, so the ward may see link previews');
 });
+
+// Playback is the heavier sibling of the preview (5.6): seeing that a video
+// exists is one decision, letting a third party's player run inside the app is
+// another. The hole this closes: the web Krant built the YouTube iframe from
+// the note's content on a path that never touched the gate, so a ward whose
+// guardians had allowed nothing still got the full player, while the app
+// showed nothing at all. The heavy thing open, the light thing shut.
+test('a player URL rides only when the playback gate is open', async () => {
+  const AP2 = (await import('../src/services/ActivityPubService.js')).default;
+  const yt = JSON.stringify({ url: 'https://www.youtube.com/watch?v=HetoL4XpHwY', title: 'x', media: [] });
+
+  const shut = AP2.timelineEmbed(yt);
+  assert.ok(shut && shut.url, 'the card itself still travels');
+  assert.equal(shut['shaer:playerUrl'], undefined, 'no player without the gate');
+
+  const open = AP2.timelineEmbed(yt, { playback: true });
+  assert.match(open['shaer:playerUrl'], /^https:\/\/www\.youtube-nocookie\.com\/embed\/HetoL4XpHwY/,
+    'privacy-enhanced only: nocookie, no related videos');
+  assert.match(open['shaer:playerUrl'], /rel=0/);
+});
+
+test('a page we will not frame simply stays a thumbnail', async () => {
+  const AP2 = (await import('../src/services/ActivityPubService.js')).default;
+  const page = JSON.stringify({ url: 'https://yougubrands.com/about', title: 'About', media: [] });
+  assert.equal(AP2.timelineEmbed(page, { playback: true })['shaer:playerUrl'], undefined);
+  assert.equal(AP2.playerUrlFor('https://nos.nl/artikel/1'), null);
+  // PeerTube is decentralised, so it is matched by shape, not by a host list.
+  assert.equal(AP2.playerUrlFor('https://tilvids.com/w/abc123def'), 'https://tilvids.com/videos/embed/abc123def');
+});
+
+test('playback needs the preview gate: you cannot play what you may not see', async () => {
+  const G2 = await import('../src/services/guardianship/index.js');
+  // Both auto (a ward): both shut.
+  assert.equal(G2.externalEmbedsAllowed(null, true), false);
+  assert.equal(G2.externalPlaybackAllowed(null, true), false);
+  // Guardians opened previews only: playback stays a separate decision.
+  assert.equal(G2.externalEmbedsAllowed(1, true), true);
+  assert.equal(G2.externalPlaybackAllowed(null, true), false);
+  // An adult account has nothing gated.
+  assert.equal(G2.externalPlaybackAllowed(null, false), true);
+});