Changeset 73a10c2 in Klonkt for src/routes/activitypub.js

Timestamp:

07/30/2026 08:40:28 AM (6 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

aa189d1

Parents:

094f7d0

Message:

Vrienden krijgen de geschiedenis mee, en een block blijft een dichte deur

Robins besluit (30-7): wie later vriend wordt, moet de oudere friends-only
posts alsnog kunnen zien. Dat kon niet: friends-only was een moment-opname
(bezorgd aan de volgers van dat moment) en de outbox verborg fan-only juist,
dus de backfill die bij een nieuw volgen draait had niets op te halen.

Drie stukken, samen de lus:

1. De outbox antwoordt naar PUBLIEK: outboxAudience beslist per lezer. De eigenaar (bearer) en een geverifieerde geaccepteerde volger of guardian krijgen de friends-only geschiedenis mee; anoniem en een vreemde met alleen een handtekening krijgen de publieke set, precies als voorheen. Bijvangst: de auteur ziet nu ook zijn EIGEN friends-only posts in de app.
2. De backfill identificeert zich: signedGetJson tekent de GET als de volgende site ((request-target) host date, wat verifyRequest checkt), dus de servende kant herkent de vriend. Een server die de handtekening negeert gedraagt zich exact als vroeger.
3. Het moment dat de vriendschap ontstaat is het moment dat de geschiedenis meekomt: op de Accept van onze Follow draait de backfill.

En Robins scherpe toevoeging: een geverifieerde caller die deze instance
BLOKKEERT krijgt een LEGE collectie, niet eens de publieke set. Een block is
een dichte deur, en een gesigneerde fetch is aankloppen met je naam erop. De
block wint ook van een verweesd volger-rijtje.

Changed files:
src/services/ActivityPubService.js

• signedGetJson; backfill gesigneerd; backfill op Accept(Follow); outboxAudience als pure, testbare beslissing

src/routes/activitypub.js

• de outbox-route beslist via outboxAudience; blocked = lege set

New file:
test/friends-history.test.js

• anoniem blijft publiek; identificatie is geen vriendschap; volger en eigenaar lezen de geschiedenis; block wint van alles, ook van een stale volger-rij

remarks: 341 tests groen, server start.

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

File:

• src/routes/activitypub.js (modified) (2 diffs)

src/routes/activitypub.js

@@ -82 +82 @@
   const site = publicSite(req.params.slug);
   if (!site) return res.status(404).end();
-  // Authorized fetch (FEP-633c §5.3 note): a committed guardian doing a SIGNED
-  // GET may read the ward's fan-only history too, without appearing as a
-  // follower. Unsigned / non-guardian callers get the public collection only.
-  let asGuardian = false;
-  if (req.headers['signature']) {
+  // Authorized fetch (30-7): who is asking decides what they see.
+  //  - the owner's own app (bearer) and a verified accepted follower or
+  //    guardian get the friends-only history too, so a NEW friend's backfill
+  //    brings the past along (Robins besluit: vrienden krijgen de
+  //    geschiedenis mee);
+  //  - a verified caller this instance BLOCKS gets an EMPTY collection, not
+  //    even the public set: a block is a closed door, and a signed fetch is
+  //    the caller knocking with their name on it;
+  //  - everyone else gets the public collection, exactly as before.
+  const bearer = OAuth.verifyBearer(req.headers.authorization);
+  let verifiedActor = null;
+  if (!bearer && req.headers['signature']) {
     const verified = await AP.verifyRequest(req).catch(() => null);
-    asGuardian = !!(verified && AP.isWardGuardian(req.params.slug, verified.id));
-  }
-  const fanClause = asGuardian ? '' : "AND (fan_only IS NULL OR fan_only = 0)";
+    verifiedActor = verified && verified.id;
+  }
+  const audience = AP.outboxAudience(req.params.slug, {
+    bearerSlug: bearer ? bearer.site.slug : null,
+    verifiedActor,
+  });
+  if (audience === 'blocked') {
+    return AP.sendAP(res, AP.buildOutbox(baseUrl(req), site, []), 'private, no-store');
+  }
+  const fanClause = audience === 'friend' ? '' : "AND (fan_only IS NULL OR fan_only = 0)";
   const posts = db.prepare(
     `SELECT id, slug, title, content, cover_image_url, cover_video_url, nsfw, content_warning, published_at, created_at
@@ -96 +110 @@
      ORDER BY COALESCE(published_at, created_at) DESC LIMIT 20`
   ).all(site.id);
-  AP.sendAP(res, AP.buildOutbox(baseUrl(req), site, posts), asGuardian ? 'private, no-store' : undefined);
+  AP.sendAP(res, AP.buildOutbox(baseUrl(req), site, posts), audience === 'friend' ? 'private, no-store' : undefined);
 });