Changeset c49808e in Klonkt

Timestamp:

08/08/2026 11:04:29 PM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

c72e45e

Parents:

cae7bfc

git-author:

Robin <roboburr@…> (08/08/2026 11:04:09 PM)

git-committer:

Robin <roboburr@…> (08/08/2026 11:04:29 PM)

Message:

actorInfo leest een url-array: de html-Link, niet de array gestringd

Barts boei-melding (8-8): een hulpvraag van @mee kwam bij @dev binnen met
mention-hrefs vol "[object Object],[object Object]". De keten: onze eigen
buildActor serveert url als ARRAY van Links (profiel + RSS, allebei
legitiem AS2), en een oudere consument op de weg van de ward stringde die
array in de href. De hedendaagse actorInfo hield met safeUrl de rommel
buiten de deur maar liet daarmee ook de profiel-URL vallen; nu kiest hij
via pickLink de html-Link, met de actor-id als terugval -- pickLink was er
nota bene al, met het commentaar "dit is de enige plek die dat weet". Dat
is nu iets minder onwaar.

De test legt het vast via de thread-byline (shaer:author), die door
dezelfde actorInfo loopt. Alle 724 groen; dev bijgewerkt.

De bezorging zelf bleek overigens NIET stuk: de hulpvraag staat gewoon in
dev's ap_mentions (help_request=1). Wat hem onvindbaar maakt is de kudde
-- 101 wards waarvan 100 daemons boven mee in elke lijst -- en dat is
Barts eigen caseload-werkstroom (shaer-6wt), niet deze fix.

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

Files:

• src/services/ActivityPubService.js (modified) (1 diff)
• test/own-card.test.js (modified) (1 diff)

src/services/ActivityPubService.js

@@ -1312 +1312 @@
   const icon = doc && doc.icon ? (doc.icon.url || (Array.isArray(doc.icon) && doc.icon[0] && doc.icon[0].url)) : null;
   const name = (doc && (doc.name || doc.preferredUsername)) || handle;
+  // Een AS2 `url` mag een ARRAY van Links zijn -- onze eigen buildActor doet
+  // dat (profiel + RSS), en een oudere consument stringde die array tot
+  // "[object Object],[object Object]" in de mention-hrefs van een hulpvraag
+  // (Barts vondst, 8-8). pickLink kiest de html-Link; de kale string blijft
+  // de gewone weg, en de actor-id de terugval.
+  const profiel = (doc && Array.isArray(doc.url))
+    ? ((pickLink(doc.url, (mt) => !mt || /html/i.test(mt)) || {}).href || safeUrl(doc.id || actorUri))
+    : safeUrl((doc && (doc.url || doc.id)) || actorUri);
   return {
     name,
     handle,
-    url: safeUrl((doc && (doc.url || doc.id)) || actorUri) || null,
+    url: profiel || null,
     icon: safeUrl(icon) || null,
     // FEP-9098 custom emojis in the display name (":shortcode:"), so the byline

test/own-card.test.js

@@ -88 +88 @@
   assert.deepEqual(await AP.previewCard('geen url'), {});
 });
+
+
+test('actorInfo kiest de html-Link uit een url-array (geen object-stringing)', async () => {
+  // Barts vondst (8-8): buildActor serveert `url` als array van Links
+  // (profiel + RSS), en een consument die dat niet kent stringde de array tot
+  // "[object Object],[object Object]" in de mention-href van een HULPVRAAG.
+  // De thread-byline loopt door dezelfde actorInfo, dus die legt het vast.
+  const HOST = 'https://203.0.113.62';
+  const NOTE = `${HOST}/notes/n1`;
+  const AUTEUR = `${HOST}/users/array`;
+  const vorige = globalThis.fetch;
+  globalThis.fetch = async (url) => {
+    const u = String(url);
+    const json = (o) => new Response(JSON.stringify(o), { status: 200, headers: { 'content-type': 'application/activity+json' } });
+    if (u === NOTE) return json({
+      id: NOTE, type: 'Note', attributedTo: AUTEUR, content: '<p>bron</p>',
+      replies: { type: 'Collection', first: { type: 'CollectionPage', items: [
+        { id: `${NOTE}/r1`, type: 'Note', attributedTo: AUTEUR, inReplyTo: NOTE, content: '<p>antwoord</p>', published: '2026-08-02T10:00:00Z' },
+      ] } },
+    });
+    if (u === AUTEUR) return json({
+      id: AUTEUR, type: 'Person', preferredUsername: 'array', inbox: `${AUTEUR}/inbox`,
+      url: [
+        { type: 'Link', href: `${HOST}/`, mediaType: 'text/html' },
+        { type: 'Link', href: `${HOST}/feed.xml`, mediaType: 'application/rss+xml' },
+      ],
+    });
+    return vorige(url);
+  };
+  const uit = await AP.getThread('schrijver', NOTE);
+  globalThis.fetch = vorige;
+  assert.equal(uit.notes.length, 1);
+  const auteur = uit.notes[0]['shaer:author'];
+  assert.equal(auteur.url, `${HOST}/`, 'de html-Link, niet de array gestringd');
+  assert.ok(!String(auteur.url).includes('object'), 'nergens [object Object]');
+});