Changeset 7d01696 in Klonkt for test/c2s-compose.test.js

Timestamp:

07/30/2026 07:56:33 AM (6 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

9ed940e

Parents:

97bcf7e

Message:

Posterframes voor video, ffmpeg-optioneel

shaer-zowq, de serverhelft. Bij een video-upload trekt ffmpeg een frame op
1 seconde naar <naam>.poster.jpg, best-effort en buiten de responspad: op een
machine zonder ffmpeg gebeurt er niets en breekt er niets (de clients halen
dan zelf een frame, native). Vanaf daar reist de poster naar drie plekken:

• poster= op de gevouwen video-tag, dus het web toont een stilstaand beeld
• opgeslagen op de c2s_attachments-entry
• als AS2 icon op het gefedereerde Video-attachment, dus apps en andere servers krijgen de thumbnail-URL cadeau

Inkomend geldt het spiegelbeeld: een remote video-attachment met een icon
houdt hem als poster in media_json, en de C2S-inboxlees serveert hem terug.

Changed files:
src/routes/activitypub.js

• uploadMedia: ffmpeg-posterframe voor video, best-effort

src/services/ActivityPubService.js

• c2sCreatePost: poster op entry en tag; buildNote: icon op het attachment; mediaFromNote/timelineAttachments: inkomende posters door

test/c2s-compose.test.js

• de poster bereikt tag, opslag en attachment; zonder posterbestand wordt er niets gegokt

remarks: 338 tests groen. ffmpeg staat NIET op de VPS; installeren is een
sudo-actie die ik niet mag doen: sudo apt-get install -y ffmpeg. Zonder dat
werkt alles, alleen maken de clients hun thumbnails zelf.

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

File:

• test/c2s-compose.test.js (modified) (1 diff)

test/c2s-compose.test.js

@@ -55 +55 @@
 });
 
+test("a video's poster frame rides the tag, the store and the federated attachment", async () => {
+  // The upload leg writes <name>.poster.jpg next to a video when ffmpeg is
+  // around (shaer-zowq). From there it must reach three places: the poster=
+  // on the folded tag (web), the stored entry, and the AS2 icon on the
+  // federated attachment (apps and other servers).
+  const os = await import('os');
+  const fsm = await import('fs');
+  const pathm = await import('path');
+  const root = fsm.mkdtempSync(pathm.join(os.tmpdir(), 'klonkt-media-'));
+  fsm.mkdirSync(pathm.join(root, 'reply-media'), { recursive: true });
+  fsm.writeFileSync(pathm.join(root, 'reply-media', 'film.mp4.poster.jpg'), 'x');
+  const prev = process.env.MEDIA_PATH;
+  process.env.MEDIA_PATH = root;
+  try {
+    const r = await AP.ingestOutboxActivity(site, user, {
+      type: 'Create',
+      object: {
+        type: 'Note', content: '<p>filmpje</p>',
+        to: ['https://test.example/ap/users/kid/followers'],
+        attachment: [{ type: 'Video', url: '/media/reply-media/film.mp4', mediaType: 'video/mp4' }],
+      },
+    });
+    assert.equal(r.status, 201);
+    const post = db.prepare('SELECT * FROM posts WHERE id = ?').get(r.id);
+    assert.match(post.content, /poster="\/media\/reply-media\/film\.mp4\.poster\.jpg"/, 'the web tag shows the still');
+    const note = AP.buildNote('https://test.example', site, post);
+    const vid = (note.attachment || []).find((a) => a.url.endsWith('film.mp4'));
+    assert.ok(vid && vid.type === 'Video');
+    assert.equal(vid.icon && vid.icon.url, 'https://test.example/media/reply-media/film.mp4.poster.jpg',
+      'the poster federates as the attachment icon');
+  } finally {
+    if (prev === undefined) delete process.env.MEDIA_PATH; else process.env.MEDIA_PATH = prev;
+  }
+});
+
+test('a video without a poster simply has none: no guessed icon', async () => {
+  const r = await AP.ingestOutboxActivity(site, user, {
+    type: 'Create',
+    object: {
+      type: 'Note', content: '<p>kaal</p>',
+      to: ['https://test.example/ap/users/kid/followers'],
+      attachment: [{ type: 'Video', url: '/media/reply-media/zonder.mp4', mediaType: 'video/mp4' }],
+    },
+  });
+  const post = db.prepare('SELECT * FROM posts WHERE id = ?').get(r.id);
+  assert.ok(!post.content.includes('poster='), 'no poster attr without a poster file');
+  const vid = (AP.buildNote('https://test.example', site, post).attachment || []).find((a) => a.url.endsWith('zonder.mp4'));
+  assert.equal(vid.icon, undefined);
+});
+
 test('a media-only post is a post, not an empty-note error', async () => {
   const r = await AP.ingestOutboxActivity(site, user, {