Changeset e27b8db in Klonkt for src/services/guardianship

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@…>

Location:

src/services/guardianship

Files:

• gated.js (modified) (1 diff)
• index.js (modified) (1 diff)
• notes.js (modified) (2 diffs)

src/services/guardianship/gated.js

@@ -56 +56 @@
 
 /** The column a feature maps onto. Unknown features are refused, not guessed. */
-const FEATURES = { 'shaer:externalEmbeds': 'external_embeds' };
+const FEATURES = {
+  'shaer:externalEmbeds': 'external_embeds',
+  'shaer:externalPlayback': 'external_playback',
+};
 export function featureColumn(feature) {
   return Object.prototype.hasOwnProperty.call(FEATURES, feature) ? FEATURES[feature] : null;

src/services/guardianship/index.js

@@ -16 +16 @@
  */
 export { SHAER_CONTEXT, GUARDIAN_RELATIONSHIP, GUARDIAN_RELATIONSHIP_COMPACT, isGuardianRelationship } from './context.js';
-export { helpRequestProps, isHelpRequest, waveProps, isWave, awayProps, hasGuardiansProps, objectHasGuardians, externalEmbedsAllowed } from './notes.js';
+export { helpRequestProps, isHelpRequest, waveProps, isWave, awayProps, hasGuardiansProps, objectHasGuardians, externalEmbedsAllowed, externalPlaybackAllowed } from './notes.js';
 export { wireDelivery, c2sVisibility, deliverDirectNote } from './delivery.js';
 export { wireHandshake, handleOutbox as handleGuardianshipOutbox, handleInbox as handleGuardianshipInbox, parseRelationship, parseUndoRelationship, endGuardianship } from './handshake.js';

src/services/guardianship/notes.js

@@ -30 +30 @@
  */
 export function externalEmbedsAllowed(setting, isWard) {
+  if (setting === 0 || setting === 1) return setting === 1;
+  return !isWard;
+}
+
+/**
+ * May a player run INSIDE the app/page for this account? The heavier sibling
+ * of the setting above, and deliberately separate: seeing that a video exists
+ * is not the same decision as handing the screen to a third party's player,
+ * with its engine, its end-screen and its next-video machine. Same shape, same
+ * default (off for a ward), and it only ever matters once embeds are allowed:
+ * you cannot play what you may not see.
+ */
+export function externalPlaybackAllowed(setting, isWard) {
   if (setting === 0 || setting === 1) return setting === 1;
   return !isWard;
@@ -74 +87 @@
 }
 
-export default { helpRequestProps, isHelpRequest, waveProps, isWave, awayProps, hasGuardiansProps, objectHasGuardians, externalEmbedsAllowed };
+export default { helpRequestProps, isHelpRequest, waveProps, isWave, awayProps, hasGuardiansProps, objectHasGuardians, externalEmbedsAllowed, externalPlaybackAllowed };