chore: track upstream SRT master fix (Haivision/srt#3354, closed 09-15) for #91, and #90 still awaits merge

Aperta
  • Qualcosa non va?
#93 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
4/5
Tempo stimato
3-5 giorni
Idoneità per principianti
35/100
Tipo di issue
Bug
Chiarezza
Abbastanza chiara
Stato di attività
Attiva
Stack tecnologico
javascript
Ambito
build-system

Direzione di ricerca

Inizia da scripts/build-srt-sdk.js e controlla il pin SRT_CHECKOUT, poi esamina la riproduzione della corruzione dell’heap di node-srt#91 e lo stato del merge di node-srt#90. Il lavoro è completato quando la riproduzione viene eseguita nuovamente su una release SRT contrassegnata che contiene cae8f624 o una versione successiva, il pin viene aggiornato se ha esito positivo e viene ottenuta la decisione del maintainer per #90.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

question ready-for-maintainer

Context

Haivision/srt#3354 (the upstream heap-corruption/hang bug behind node-srt#91, "AsyncSRT.dispose()/
worker.terminate() causes native heap corruption") was closed 2026-09-15. The maintainer's comment thread
confirms: rebuilt against current master (cae8f624), the same repro suite that previously crashed reliably ran
6/6 clean.

The gap

The latest tagged release, v1.5.7 (published 2026-08-28), predates the 09-15 fix confirmation on
master. No tag yet contains the fix, so node-srt#91 cannot be closed by re-pinning to an existing release.

Separately, node-srt#90 (bump bundled libSRT v1.5.5 -> v1.5.6, fixing CVE-2026-55869 / CVE-2026-55868) is
unrelated to #91 and has sat open since 2026-08-13: mergeable: MERGEABLE, mergeStateStatus: UNSTABLE, and
is explicitly flagged in its own body as a "Tier B" native-build change awaiting a human maintainer rather than
auto-merge, even with the CVE fix verified.

Ask

  1. Track Haivision/srt for the next tagged release containing cae8f624 (or later); when it lands, re-verify
    #91's repro against it and update scripts/build-srt-sdk.js's SRT_CHECKOUT pin accordingly.
  2. Separately, #90 is CI-green and CVE-verified; it just needs the named human merge decision it has been
    waiting 35 days for.

Actionability

  • Who can act: Birme's team (merge #90 now; watch for the next SRT tag for #91).
  • Type of work: chore / dependency tracking
  • Effort: S (tracking note + a merge decision)
  • Blocked by: upstream tag cut for #91's half; nothing for #90's half.
  • Demo-critical: No.
Lingua principale
JavaScript
Stelle
99
Fork
35
Nessuna PR unita negli ultimi 30g

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di Eyevinn/node-srt

Tutte le issue di Eyevinn/node-srt

Issue simili

Altre issue su JavaScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.