Changeset b1512e0 in Klonkt for src/services

Timestamp:

07/28/2026 08:54:20 AM (6 weeks ago)

Author:

Robin Genis <roboburr@…>

Branches:

main

Children:

bdb78e4

Parents:

3f32994

Message:

De kop-lezer stopte op een og:image in een script, net voor de echte tag

Op mijn machine werkte YouTube, op de VPS niet. Verschil: die krijgt een andere
variant van de pagina (lang=de-DE), en daarin staat de string og:image eerst in
inline JavaScript. Mijn vroege stop trapte daarin en kapte de pagina af op 661kB,
net voor het echte meta-blok. Resultaat: geen oEmbed-link, geen og-tags, geen
kaart.

Dat is dezelfde near-miss als de body-cap eerder, met een andere oorzaak: ik
stopte met lezen op het moment dat het ER uitzag alsof ik klaar was. De stop
vraagt nu om een echte <meta ...og:image en niet om de kale string.

Changed files:
src/services/EmbedResolver.js

• vroege stop vereist een meta-tag, geen losse string

src/services/ActivityPubService.js

• SELFHEAL_VERSION 18 -> 19

test/embed-resolver.test.js

• regressietest met een lokkertje in een script voor de echte tag

remarks: 215 tests groen.

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

Location:

src/services

Files:

• ActivityPubService.js (modified) (1 diff)
• EmbedResolver.js (modified) (1 diff)

src/services/ActivityPubService.js

@@ -2849 +2849 @@
 // during a flux window, e.g. a fleet-wide update), and drops notes that are gone
 // (404/410). Bump SELFHEAL_VERSION only on a release that warrants a re-sync.
-const SELFHEAL_VERSION = 18; // v18: link previews no longer hang behind the note re-fetch (an unreachable origin skipped the whole row)
+const SELFHEAL_VERSION = 19; // v19: head-read no longer stops on an og:image mention inside inline script
 async function fetchNoteAP(url) {
   try {

src/services/EmbedResolver.js

@@ -231 +231 @@
       // the full string every read turns a 1MB page into quadratic work.
       const window = tail + chunk;
-      if (len >= MAX_HEAD || /<\/head>/i.test(window) || /og:image/i.test(window)) done_ = true;
+      // Stop on the real <meta property="og:image">, not on the bare string.
+      // Big sites carry "og:image" inside inline JSON long before the actual
+      // tag, and stopping there cut the page off just short of the meta block:
+      // the same near-miss as the old size cap, with a different cause.
+      if (len >= MAX_HEAD || /<\/head>/i.test(window) || /<meta[^>]{0,300}og:image/i.test(window)) done_ = true;
       tail = chunk.slice(-512);
     }