source: Klonkt/test/external-embeds.test.js@ fc40410

main
Last change on this file since fc40410 was fc40410, checked in by Robin Genis <roboburr@…>, 6 weeks ago

Externe embeds: thumbnail-only, en gated aan de serverkant

Fase 2 van shaer-277. Een note zonder fediverse-quote maar met een externe link
levert nu een embed-kaart op, via dezelfde resolver (oEmbed of een bekende
provider). Twee besluiten zitten erin verankerd:

THUMBNAIL-ONLY. Nooit de iframe van de aanbieder. Een willekeurig
derde-partij-frame in een kindveilige app is een gat dat je niet meer dicht
krijgt, dus de kaart draagt een afbeelding en een titel en niets uitvoerbaars.

GATED AAN DE SERVERKANT. Externe embeds zijn een gated feature: de wereld van
een ward buiten de fediverse is aan de guardians. Cruciaal is WAAR die gate zit:
bij het serialiseren, niet in de client. Een embed die de client alleen maar
verbergt, is wel degelijk al op het toestel afgeleverd. Staat de gate dicht, dan
gaat shaer:embed simpelweg niet mee.

De regel zelf (externalEmbedsAllowed) is puur en apart getest: null = auto, wat
uit staat voor een ward en aan voor ieder ander; een expliciet guardian-besluit
wint beide kanten op.

Changed files:
src/config/database.js

  • ap_timeline.embed_json en sites.external_embeds (NULL = auto)

src/services/guardianship/notes.js

  • externalEmbedsAllowed(setting, isWard), puur en geexporteerd

src/services/guardianship/index.js

  • doorgeexporteerd

src/services/ActivityPubService.js

  • resolveExternalEmbed + firstExternalUrl + timelineEmbed
  • inbound: geen quote maar wel een externe link -> embed resolven (out of band)

src/routes/activitypub.js

  • inbox-read past de gate toe en serveert shaer:embed alleen als die open staat

New file:
test/external-embeds.test.js

  • 5 tests: de auto-regel, het expliciete besluit, link-selectie (mentions en hashtags overslaan), en dat een kaart zonder url geen kaart is

remarks: 205 tests groen. Nog te doen: de clients laten shaer:embed renderen met
de bestaande QuoteCard, en een UI voor guardians om de gate te bedienen (hoort
bij shaer-3kp).

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

  • Property mode set to 100644
File size: 2.4 KB
RevLine 
[fc40410]1// External embeds are a gated feature (FEP-633c): a ward's world outside the
2// fediverse is the guardians' call, and the gate is applied server-side.
3import { test } from 'node:test';
4import assert from 'node:assert/strict';
5process.env.DATABASE_PATH = ':memory:';
6process.env.PUBLIC_BASE_URL = 'https://test.example';
7const dbMod = await import('../src/config/database.js');
8dbMod.initializeDatabase();
9const { externalEmbedsAllowed } = await import('../src/services/guardianship/notes.js');
10const { firstExternalUrl, timelineEmbed } = await import('../src/services/ActivityPubService.js');
11
12test('auto (no setting): off for a ward, on for anyone else', () => {
13 assert.equal(externalEmbedsAllowed(null, true), false, 'a ward gets no external embeds by default');
14 assert.equal(externalEmbedsAllowed(null, false), true, 'a free actor does');
15 assert.equal(externalEmbedsAllowed(undefined, true), false);
16});
17
18test('an explicit guardian decision wins over the default, both ways', () => {
19 assert.equal(externalEmbedsAllowed(1, true), true, 'guardians may open it for a ward');
20 assert.equal(externalEmbedsAllowed(0, false), false, 'and may close it for anyone');
21});
22
23test('firstExternalUrl picks the first real link, skipping mentions and hashtags', () => {
24 const html = '<p><a href="https://s/@bob" class="u-url mention">@bob</a> '
25 + '<a href="https://s/tags/x" class="mention hashtag">#x</a> '
26 + 'kijk: <a href="https://v.example/watch/1">dit</a> en <a href="https://later.example/">dat</a></p>';
27 assert.equal(firstExternalUrl(html), 'https://v.example/watch/1');
28});
29
30test('firstExternalUrl ignores non-http hrefs and empty content', () => {
31 assert.equal(firstExternalUrl('<a href="javascript:alert(1)">x</a>'), null);
32 assert.equal(firstExternalUrl('<p>geen links</p>'), null);
33 assert.equal(firstExternalUrl(null), null);
34});
35
36test('timelineEmbed round-trips a stored card and refuses junk', () => {
37 const stored = JSON.stringify({ url: 'https://v.example/1', kind: 'oembed', title: 'A talk', media: [{ url: 'https://v.example/t.jpg' }] });
38 const back = timelineEmbed(stored);
39 assert.equal(back.title, 'A talk');
40 assert.equal(back.media[0].url, 'https://v.example/t.jpg');
41 assert.equal(timelineEmbed(null), undefined);
42 assert.equal(timelineEmbed('not json'), undefined);
43 assert.equal(timelineEmbed('{"title":"no url"}'), undefined, 'a card without a url is not a card');
44});
Note: See TracBrowser for help on using the repository browser.