Changeset 0517013 in Klonkt

Timestamp:

08/08/2026 04:15:57 AM (5 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

553bd70

Parents:

06bae1e

git-author:

Robin <roboburr@…> (08/08/2026 04:15:44 AM)

git-committer:

Robin <roboburr@…> (08/08/2026 04:15:57 AM)

Message:

De Accept-terugval werkt nu ook op de gedeelde inbox

Vervolg op 06bae1e, en de fout die daar in zat: de terugval hing aan
slugParam, en die is LEEG wanneer er op /ap/inbox wordt afgeleverd in plaats
van op /ap/users/<slug>/inbox. Funkwhale doet precies dat, dus de terugval
werd overgeslagen en de rij bleef alsnog op pending -- alleen zei de log nu
tenminste eerlijk "accept UNMATCHED".

Wie wij zijn staat gewoon in de ingesloten Follow: die hebben wij zelf
verstuurd, dus object.actor is onze eigen actor-URI. Daar leiden we de slug
uit af als slugParam ontbreekt.

Zonder bruikbare Follow raden we niets: dan blijft het pending. Er is een test
die dat vastlegt, naast de test voor de gedeelde-inbox-weg zelf.

7 tests.

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

Files:

• src/services/ActivityPubService.js (modified) (1 diff)
• test/follow-accept-match.test.js (modified) (1 diff)

src/services/ActivityPubService.js

@@ -2523 +2523 @@
     // rij die nog op pending staat wordt geraakt, dus dit kan niets anders
     // openzetten dan een follow die wij zelf hebben verstuurd.
-    if (!raak && slugParam && actorUri) {
-      try { raak = fwStmts().accByActor.run(slugParam, actorUri).changes; } catch { /* ignore */ }
+    //
+    // En de slug mag NIET van slugParam afhangen: Funkwhale bezorgt op de
+    // GEDEELDE inbox, en dan is die leeg. Wie wij zijn staat in de ingesloten
+    // Follow -- die hebben wij immers zelf verstuurd, dus `object.actor` is
+    // onze eigen actor-URI.
+    let mij = slugParam;
+    if (!mij && act.object && typeof act.object === 'object') mij = slugFromActorUrl(act.object.actor);
+    if (!raak && mij && actorUri) {
+      try { raak = fwStmts().accByActor.run(mij, actorUri).changes; } catch { /* ignore */ }
     }
     // Eerlijk loggen: zonder treffer is er niets geaccepteerd, en dat hoort te

test/follow-accept-match.test.js

@@ -64 +64 @@
 });
 
+test('bezorgd op de GEDEELDE inbox: de slug komt uit de ingesloten Follow', async () => {
+  // Funkwhale levert af op /ap/inbox, niet op /ap/users/<slug>/inbox. Dan is
+  // slugParam null en moeten we uit `object.actor` afleiden wie wij zijn --
+  // die Follow hebben wij immers zelf verstuurd.
+  zetPending();
+  await AP.handleInbox({
+    body: {
+      type: 'Accept', actor: REMOTE,
+      object: {
+        id: 'https://test.example/ap/users/dev#follows/389cf422-7e38-4bad-858e-35def4c3e2e9',
+        type: 'Follow', actor: 'https://test.example/ap/users/dev', object: REMOTE,
+      },
+    },
+    headers: {}, get: () => undefined, socket: {},
+  }, null, { id: REMOTE });
+  assert.equal(status(), 'accepted');
+});
+
+test('op de gedeelde inbox zonder bruikbare Follow blijft het pending', async () => {
+  zetPending();
+  await AP.handleInbox({
+    body: { type: 'Accept', actor: REMOTE, object: { id: 'https://elders.example/x', type: 'Follow' } },
+    headers: {}, get: () => undefined, socket: {},
+  }, null, { id: REMOTE });
+  assert.equal(status(), 'pending', 'zonder te weten wie wij zijn, raden we niet');
+});
+
 test('een Accept van een ANDERE actor raakt onze rij niet', async () => {
   zetPending();