Changeset 32a4ffb in Klonkt

Timestamp:

08/06/2026 01:27:37 PM (5 weeks ago)

Author:

roboburr <roboburr@…>

Branches:

main

Children:

621126b

Parents:

279ca0b

git-author:

Robin <roboburr@…> (08/06/2026 01:27:28 PM)

git-committer:

roboburr <roboburr@…> (08/06/2026 01:27:37 PM)

Message:

Onthoud de URI van antwoorden die we al ontvangen (shaer-e9g)

Antwoorden van accounts die je volgt komen gewoon binnen, ondertekend door de
schrijver zelf. belongsInTimeline houdt ze uit de Krant -- een antwoord hoort in
zijn draad, niet in de tijdlijn -- en daarna bewaarden we ze nergens. Kwam er
later een DOORGESTUURD antwoord op zo'n bericht, dan kende knownNoteUri de ouder
niet en werd het afgewezen. Terwijl we die ouder wel degelijk hadden gehad.

Op boiert ging daar 72% van het doorstuurverkeer op stuk: 18 unieke reacties
geweigerd tegen 7 geaccepteerd, in drie uur. Alle achttien waren echte antwoorden
in echte gesprekken, en de ontbrekende ouders waren stuk voor stuk geschreven
door accounts die boiert volgt.

Nieuwe tabel ap_seen_notes, alleen de URI. Geen inhoud, niets extra's op het
scherm, geen tweede administratie naast ap_timeline: dit beantwoordt uitsluitend
de vraag 'kennen wij dit bericht?'. Na 30 dagen gesnoeid, af en toe tijdens het
schrijven zodat een server die weken doorloopt ook opruimt -- doorsturen gebeurt
kort na het antwoord.

De grens zit in WIE er onthouden wordt, en dat is de hele veiligheidsvraag.
Alleen schrijvers die je zelf volgt. Zou het voor iedereen gelden, dan kan een
vreemde eerst een bericht neerleggen en daarna met een doorgestuurd antwoord
daarop de dereference naar een adres van zijn keuze sturen. Er wordt dus niets
van een onbekende aangenomen dat we niet al hadden.

Dit volgt op Barts keuze om gesprekken van anderen NIET te gaan volgen: dat
blijft zo. We stoppen alleen met weggooien wat we al kregen.

Gecontroleerd dat de tests bijten: haal de lookup uit knownNoteUri en geval 5
valt om, haal de volg-eis weg en geval 3. Suite 478/478.

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

Files:

• src/config/database.js (modified) (1 diff)
• src/services/ActivityPubService.js (modified) (2 diffs)
• test/seen-notes.test.js (added)

src/config/database.js

@@ -469 +469 @@
       UNIQUE(slug, actor_uri)
     );
+    -- Antwoorden van accounts die we volgen komen gewoon binnen, ondertekend
+    -- door de schrijver, maar horen niet in de Krant (belongsInTimeline) en
+    -- werden daarna nergens bewaard. Kwam er later een doorgestuurd antwoord OP
+    -- zo'n bericht, dan kenden we de ouder niet en wezen we het af (shaer-e9g).
+    -- Alleen de URI, geen inhoud: dit voedt uitsluitend de vraag "kennen wij dit
+    -- bericht?". Wordt na 30 dagen gesnoeid; doorsturen gebeurt kort na het
+    -- antwoord, dus langer bewaren levert niets op.
+    CREATE TABLE IF NOT EXISTS ap_seen_notes (
+      uri TEXT PRIMARY KEY,
+      created_at DATETIME DEFAULT CURRENT_TIMESTAMP
+    );
+    CREATE INDEX IF NOT EXISTS idx_ap_seen_notes_age ON ap_seen_notes(created_at);
     CREATE TABLE IF NOT EXISTS ap_timeline (
       id TEXT NOT NULL,              -- the remote note's AP id

src/services/ActivityPubService.js

@@ -1715 +1715 @@
     if (db.prepare('SELECT 1 FROM ap_timeline WHERE id = ? LIMIT 1').get(uri)) return true;
     if (db.prepare('SELECT 1 FROM ap_interactions WHERE object_uri = ? LIMIT 1').get(uri)) return true;
+    // Een antwoord dat we al bezorgd kregen van iemand die we volgen (shaer-e9g).
+    if (db.prepare('SELECT 1 FROM ap_seen_notes WHERE uri = ? LIMIT 1').get(uri)) return true;
   } catch { /* bij twijfel niet ophalen */ }
   return false;
 }
+
+/**
+ * Onthoud dat we dit bericht al eens bezorgd kregen.
+ *
+ * Alleen de URI. Geen inhoud, niets op het scherm, geen tweede weergave -- dit
+ * beantwoordt uitsluitend de vraag "kennen wij dit bericht?" die knownNoteUri
+ * stelt voordat er iets bij de bron wordt opgehaald.
+ *
+ * De beller bepaalt WIE er onthouden wordt, en dat is de hele veiligheidsvraag:
+ * onthouden we zomaar alles wat iemand aflevert, dan kan een vreemde eerst een
+ * bericht neerleggen en daarna met een doorgestuurd antwoord dáárop ons naar een
+ * adres van zijn keuze sturen. Vandaar dat handleInbox dit alleen doet voor
+ * schrijvers die je zelf volgt.
+ */
+const SEEN_NOTES_DAYS = 30;
+let _seenSinceSnoei = 0;
+function rememberNoteUri(uri) {
+  if (!uri || typeof uri !== 'string') return;
+  try {
+    db.prepare('INSERT OR IGNORE INTO ap_seen_notes (uri) VALUES (?)').run(uri);
+    // Af en toe opruimen, niet bij het opstarten: een server die weken doorloopt
+    // zou anders nooit snoeien. Doorsturen gebeurt kort na het antwoord, dus wat
+    // ouder is dan een maand beantwoordt geen enkele vraag meer.
+    if (++_seenSinceSnoei >= 500) {
+      _seenSinceSnoei = 0;
+      const r = db.prepare(`DELETE FROM ap_seen_notes WHERE created_at < datetime('now', '-${SEEN_NOTES_DAYS} days')`).run();
+      if (r.changes) console.log(`[AP] seen notes: ${r.changes} pruned`);
+    }
+  } catch { /* niet fataal */ }
+}
+const isFollowedActor = (uri) => {
+  try { return !!db.prepare('SELECT 1 FROM ap_following WHERE actor_uri = ? LIMIT 1').get(uri); } catch { return false; }
+};
 
 // Mislukte dereferences kort onthouden. Mastodon herhaalt een bezorging
@@ -2124 +2159 @@
         console.log('[AP] timeline +', actorUri, 'x' + subs.length);
       }
+    }
+    // Een ANTWOORD van iemand die we volgen: bewaar de URI (shaer-e9g). Zo'n
+    // bericht komt hier gewoon binnen, ondertekend door de schrijver zelf, maar
+    // belongsInTimeline houdt het uit de Krant en daarna raakten we het kwijt.
+    // Kwam er later een doorgestuurd antwoord OP dat bericht, dan kenden we de
+    // ouder niet en wezen we het af -- terwijl we hem wel degelijk hadden gehad.
+    // Er verandert niets aan wat we tonen of van vreemden aannemen: de schrijver
+    // moet iemand zijn die je zelf bent gaan volgen.
+    if (actorUri && !isLocalActor && o.id && o.inReplyTo && noteVisibility(o) !== 'direct' && isFollowedActor(actorUri)) {
+      rememberNoteUri(o.id);
     }
     // Mentioned in a post that is NOT a reply to our content (a reply to us already returned