Changeset e27b8db in Klonkt for src/services/ActivityPubService.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:

• src/services/ActivityPubService.js (modified) (2 diffs)

src/services/ActivityPubService.js

@@ -3043 +3043 @@
 
 /** The stored external-embed card, for the C2S read. */
-export function timelineEmbed(embedJson) {
-  try { const e = embedJson ? JSON.parse(embedJson) : null; return (e && typeof e === 'object' && e.url) ? e : undefined; }
-  catch { return undefined; }
+export function timelineEmbed(embedJson, { playback = false } = {}) {
+  try {
+    const e = embedJson ? JSON.parse(embedJson) : null;
+    if (!e || typeof e !== 'object' || !e.url) return undefined;
+    // The player URL is served ONLY when the playback gate is open (FEP-633c
+    // 5.6). Deciding it here keeps the provider knowledge in one place: the
+    // client never needs a list of hosts, it just plays what it is handed.
+    // Privacy-enhanced variants only: nocookie for YouTube, the instance's own
+    // player for PeerTube. Without one the card stays a thumbnail.
+    const player = playback ? playerUrlFor(e.url) : null;
+    return player ? { ...e, 'shaer:playerUrl': player } : e;
+  } catch { return undefined; }
+}
+
+/** The embeddable player for a URL, or null when we will not frame it. */
+export function playerUrlFor(url) {
+  if (typeof url !== 'string') return null;
+  let p = null;
+  try { p = AudioEmbedService.detectProvider(url); } catch { p = null; }
+  if (p && p.provider === 'youtube' && p.id) return `https://www.youtube-nocookie.com/embed/${p.id}?rel=0&modestbranding=1&playsinline=1`;
+  if (p && p.provider === 'vimeo' && p.id) return `https://player.vimeo.com/video/${p.id}`;
+  // PeerTube is decentralised, so it is matched by its watch-URL shape rather
+  // than a provider list. Host chars are validated before it is inlined.
+  const pt = url.match(/^https?:\/\/([\w.-]+(?::\d+)?)\/(?:w|videos\/watch)\/([\w-]{6,})/i);
+  if (pt) return `https://${pt[1]}/videos/embed/${pt[2]}`;
+  return null;
 }
 
@@ -3877 +3900 @@
   getReplyUris, markNotificationsSeen, countUnseenNotifications, hasPlayableAudio,
   linkifyBody, bakePostContent, bakePostContentWithMentions, listFollowers, removeFollower, listConnections,
-  noteVisibility, belongsInTimeline, isRejectedObject, rejectInteraction, interactionReportTarget,
+  noteVisibility, belongsInTimeline, playerUrlFor, isRejectedObject, rejectInteraction, interactionReportTarget,
   getMessages, notificationsSeenAt, ingestOutboxActivity, c2sVisibility, actorDisplay, buildActorRef, prefersEnriched,
 };