Changeset b1512e0 in Klonkt for src/services/EmbedResolver.js

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@…>

File:

• src/services/EmbedResolver.js (modified) (1 diff)

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);
     }