When `ReadableStream` read fails, stack trace does not include the line responsible
Nessuno ha ancora preso questa issue.
- Lingua principale
- JavaScript
- Stelle
- 122k
- Fork
- 37.3k
- Merge medio
- 4g 2h
- PR unite (30g)
- 283
Descrizione
Version
22.2.0
Platform
Microsoft Windows NT 10.0.22635.0 x64
Subsystem
No response
What steps will reproduce the bug?
Save the following code to a file and run it:
import net from 'node:net'
import stream from 'node:stream'
const socket = net.connect(90, 'host.invalid.',)
socket.on("error", e => { console.warn("socket error:", e.message) })
let myStream = stream.Duplex.toWeb(socket)
console.log("next line throws")
await myStream.readable.getReader().read()
console.log("this line will never be reached")
How often does it reproduce? Is there a required condition?
No response
What is the expected behavior? Why is that the expected behavior?
From the output, you can see there was an error, but you can't see why that crashed the process. The output should mention the line number of the read() call so that the user can add necessary error handling.
Note that --async-stack-traces does not make the output any more useful.
What do you see instead?
> node index.mjs
next line throws
socket error: getaddrinfo ENOTFOUND host.invalid.
node:internal/modules/run_main:115
triggerUncaughtException(
^
Error: getaddrinfo ENOTFOUND host.invalid.
at GetAddrInfoReqWrap.onlookupall [as oncomplete] (node:dns:120:26) {
errno: -3008,
code: 'ENOTFOUND',
syscall: 'getaddrinfo',
hostname: 'host.invalid.'
}
Additional information
A bad and easy to forget workaround is to wrap each await call.
await reader.read().catch(e => { throw new AggregateError([e]) })
A better workaround might be possible with the (discouraged) async hooks API.
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia eseguendo la riproduzione fornita in index.mjs con Node.js 22.2.0 e analizza il reader di ReadableStream e la propagazione degli errori coinvolta in stream.Duplex.toWeb(socket). Il lavoro è completato quando l'errore non intercettato include la posizione nel sorgente della chiamata a read(), con un test di regressione che copra il percorso read che fallisce.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- javascript, node.js
- Ambito
- backend
- Tipo di issue
- Bug
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Tranquilla
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 45/100