Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

Windows ConPTY: unhandled 'error' event on conout/conin sockets + re-throw in error handler kills the embedding host

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

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
3/5
Tempo stimato
1-2 giorni
Idoneità per principianti
72/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
node.js, typescript
Ambito
backend, operating-systems

Direzione di ricerca

Inizia in lib/windowsPtyAgent.js, intorno alla costruzione di _outSocket e _inSocket, quindi segui il percorso _failPtyConnection esistente. Leggi la gestione degli errori dei socket in lib/windowsTerminal.js e verifica come ready_datapipe e agent.onError portano alla pulizia. Il lavoro è completato quando le pipe ConPTY danneggiate non producono più un'eccezione non gestita né terminano l'host che le integra; la verifica riportata ha usato node --check.

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

Descrizione

Summary

Windows ConPTY (named pipe) sockets created in WindowsPtyAgent have no 'error' listener at construction time, and WindowsTerminal's error handler re-throws non-EIO errors from inside the 'error' event. Both paths turn a broken ConPTY pipe into an uncaught exception that kills the whole host process (observed twice, e.g. exit code 1 with the classic Unhandled 'error' event dump and once with 0xC0000005).

Version: 1.2.0-beta.15 on Windows (Node v24.14.0), consumed via dsh-subprocess-local.spawnTerminal.

Repro / trigger

Any long-running persistent PTY whose ConPTY pipe breaks while the host is alive:

  • child process tree killed externally (e.g. taskkill /T, cleanup tools, watchdog scripts)
  • machine sleep/wake invalidating the pipe
  • connection timeout / worker failure before ready_datapipe

Once the pipe breaks, the next socket read/write fails with Error: read UNKNOWN (errno: -4094, code: 'UNKNOWN') and the whole embedding process dies.

Evidence

Host-process stderr captured at crash:

node:events:486
      throw er; // Unhandled 'error' event
      ^

Error: read UNKNOWN
    at Pipe.onStreamRead (node:internal/stream_base_commons:216:20)
Emitted 'error' event on Socket instance at:
    at emitErrorNT (node:internal/streams/destroy:170:8)
    at emitErrorCloseNT (node:internal/streams/destroy:129:3)
    at process.processTicksAndRejections (node:internal/process/task_queues:90:21) {
  errno: -4094,
  code: 'UNKNOWN',
  syscall: 'read'
}

Node.js v24.14.0

Root cause (code)

lib/windowsPtyAgent.js (constructor, ~lines 56-93):

this._outSocket = new net_1.Socket();
...
this._outSocket.on('connect', function () {
    _this._outSocket.emit('ready_datapipe');
});
var inSocketFD = fs.openSync(term.conin, 'w');
this._inSocket = new net_1.Socket({ fd: inSocketFD, readable: false, writable: true });
  • _outSocket: the 'error' listener is only attached later in windowsTerminal.js inside the ready_datapipe callback — before that fires, an error on this socket is unhandled → process crash.
  • _inSocket: never gets an 'error' listener at all → any write/read error = Unhandled 'error' event → process crash.

lib/windowsTerminal.js (~lines 89-105):

_this._socket.on('error', function (err) {
    _this._close();
    if (err.code) {
        if (~err.code.indexOf('errno 5') || ~err.code.indexOf('EIO')) return;
    }
    // Throw anything else.
    if (_this.listeners('error').length < 2) {
        throw err;   // throw from inside an 'error' handler = uncaught exception
    }
});

Suggested fix

  1. Attach 'error' handlers on both sockets immediately after creation in WindowsPtyAgent, funneling into the existing _failPtyConnection path (which fires onError → terminal exit → graceful cleanup).
  2. Remove/replace the throw err in WindowsTerminal — errors already flow via agent.onError → exit; re-throwing from an 'error' listener is a process-killer for embedding consumers.

I applied exactly these two changes locally (verified with node --check) and the host process no longer dies on the broken-pipe path.

Lingua principale
TypeScript
Stelle
2k
Fork
337
Merge medio
13h 17m
PR unite (30g)
5

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 microsoft/node-pty

Tutte le issue di microsoft/node-pty

Issue simili

Altre issue su TypeScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.