Changeset 4425cc2 in Klonkt for src/routes/activitypub.js

Timestamp:

09/04/2026 11:36:18 AM (4 days ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

369aa60

Parents:

59f404a

Message:

Featured: de kolommen die buildNote nodig heeft (shaer-6oth, laatste punt)

De bead vroeg nog na te gaan of dezelfde kolommen ergens anders ontbraken
waar buildNote op een DB-rij wordt losgelaten. Alle acht aanroeppunten
langsgelopen: zes zijn schoon (twee lezen SELECT *, backfillNewFollower is
15-8 al bijgewerkt, en de drie handgebouwde rijen in de webroutes geven de
velden mee). De featured-collectie was het gat.

De gok in de bead was half raak. Featured filtert wel op fan_only --
daarom leek hij goed -- maar selecteert ap_visibility, paid en excerpt
niet, en filtert daar ook niet op. Twee gevolgen, op een route die
publiek en onbetekend opvraagbaar is:

een vastgezette BETAALDE post lekte zijn volledige tekst, want
buildNote slaat zijn redactie over zodra post.paid undefined is. Precies
het mechanisme van Barts melding van 15-8; toen zijn outboxSlice en
backfillNewFollower bijgewerkt en is deze route gemist.

een vastgezette QUIET-post kreeg to: as:Public in plaats van zijn
volgers -- luider dan de schrijver koos, dezelfde vorm als shaer-6oth.

excerpt hoort erbij, anders valt de teaser na de reparatie terug op de
eerste alinea van precies de tekst die verborgen moet blijven. Het
visibility-filter erbij: niet-publiek hoort niet in een publieke
collectie, ook niet vastgezet.

De toets is een LUS over de publieke collecties die posts dragen, niet
een toets op deze ene query -- dezelfde reden die ap-collection-paging
al noemt: het gat ontstond doordat de ene plek het wel had en de andere
niet, en een toets per plek vraagt om dezelfde vergeetachtigheid. Over
HTTP, want de query zit in de route.

Tegenbewijs: met de kolommen er weer uit vallen precies de twee
featured-toetsen (het lek is dus gemeten, niet beredeneerd), blijft de
outbox groen omdat die al gerepareerd was, en blijft de vrienden-toets
groen -- wat bevestigt dat het fan_only-filter er altijd al was. Eerste
poging tot dat tegenbewijs sloeg niet aan doordat de mutatie niet
matchte; die is overgedaan met controle dat hij er echt in zat.

Volle suite 1245 groen.

File:

• src/routes/activitypub.js (modified) (1 diff)

src/routes/activitypub.js

@@ -1153 +1153 @@
   // last-processed-first). So we emit it reversed (lowest pin priority first,
   // rank 1 last) → Mastodon flips it back to pin-rank ascending on the profile.
+  // fan_only, ap_visibility, paid en excerpt MOETEN mee, om dezelfde reden als
+  // in outboxSlice en backfillNewFollower (shaer-6oth, en Barts melding van
+  // 15-8). buildNote beslist op deze velden, en een ontbrekende kolom is daar
+  // `undefined` -- wat stilletjes het ruimste gedrag oplevert:
+  //
+  //   zonder `paid`    slaat buildNote zijn redactie over en gaat de VOLLEDIGE
+  //                    tekst van een betaalde post mee. Deze collectie is
+  //                    publiek en onbetekend opvraagbaar, dus dat is de post
+  //                    gewoon te lezen. `excerpt` hoort erbij, anders valt de
+  //                    teaser terug op de eerste alinea van precies de tekst
+  //                    die verborgen moet blijven.
+  //   zonder ap_vis    krijgt een quiet/unlisted post `to: as:Public` in plaats
+  //                    van zijn volgers -- luider dan de schrijver koos.
+  //
+  // Het filter erbij: fan_only stond er al, ap_visibility ontbrak. Een post die
+  // niet publiek bedoeld is hoort niet in een publieke collectie, ook niet als
+  // hij vastgezet is.
   const posts = db.prepare(
-    `SELECT id, slug, title, content, cover_image_url, cover_video_url, nsfw, content_warning, c2s_attachments, published_at, created_at
+    `SELECT id, slug, title, content, cover_image_url, cover_video_url, nsfw, content_warning, c2s_attachments, published_at, created_at,
+            fan_only, ap_visibility, paid, paid_min_cents, excerpt
      FROM posts WHERE site_id = ? AND status = 'published' AND (fan_only IS NULL OR fan_only = 0)
+       AND IFNULL(ap_visibility, 'public') IN ('public', 'quiet')
        AND pinned IS NOT NULL AND pinned > 0
      ORDER BY pinned DESC, COALESCE(published_at, created_at) ASC LIMIT 20`