Changeset b1512e0 in Klonkt for test/embed-resolver.test.js


Ignore:
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:
1 edited

Legend:

Unmodified
Added
Removed

  • test/embed-resolver.test.js

    r3f32994 rb1512e0  
    182182  assert.ok(page && page.includes('og:image'), 'the head survives the cap');
    183183});
     184
     185// Regression: big sites carry the bare string "og:image" inside inline JSON long
     186// before the real meta tag. Stopping the read there cut the page off just short
     187// of the tags and produced no card at all.
     188test('the head read does not stop on an og:image mention inside a script', async () => {
     189  const { liveIO } = await import('../src/services/EmbedResolver.js');
     190  const page = '<html><head><script>var cfg={"og:image":"decoy"};</script>'
     191    + 'y'.repeat(3000)
     192    + '<meta property="og:title" content="Echt"><meta property="og:image" content="https://x/real.png">'
     193    + '</head></html>';
     194  const fakeFetch = async () => ({ ok: true, headers: { get: () => '999' }, text: async () => page });
     195  const io = liveIO({ safeFetch: fakeFetch, detectProvider: () => null });
     196  const got = await io.getPage('https://x/p');
     197  const { findOpenGraph } = await import('../src/services/EmbedResolver.js');
     198  assert.equal(findOpenGraph(got).image, 'https://x/real.png', 'reads past the decoy to the real tag');
     199});
Note: See TracChangeset for help on using the changeset viewer.