When `ReadableStream` read fails, stack trace does not include the line responsible
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- JavaScript
- Estrellas
- 122k
- Forks
- 37.3k
- Merge medio
- 4 d 2 h
- PR fusionados (30 d)
- 283
Descripción
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.
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Comienza ejecutando la reproducción proporcionada en index.mjs con Node.js 22.2.0 e inspecciona el lector de ReadableStream y la propagación de errores implicada en stream.Duplex.toWeb(socket). Se considera completado cuando el error no capturado incluye la ubicación de origen de la llamada a read(), con una prueba de regresión que cubra la ruta de read que falla.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- javascript, node.js
- Área
- backend
- Tipo de issue
- Error
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Tranquilo
- Claridad
- Bastante claro
- Aptitud para principiantes
- 45/100