⚡ Sangsakala Proxy 🏠 Home
Skip to content

deps: backport libuv TTY hangup fix - #67237

Open
alexforster wants to merge 1 commit into
nodejs:mainfrom
alexforster:deps-backport-libuv-pty-hangup
Open

alexforster wants to merge 1 commit into
nodejs:mainfrom
alexforster:deps-backport-libuv-pty-hangup

Conversation

@alexforster

@alexforster alexforster commented Nov 4, 2026 •

Copy link
Copy Markdown

PTY output can be truncated when a hangup arrives before all buffered data has been read. I ran into this while using a terminal emulator library.

Asking for a small, targeted backport of this upstream libuv fix to read the remaining data before EOF.

Refs: libuv/libuv#5165
Signed-off-by: Alex Forster alex@alexforster.com

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/security-wg

@nodejs-github-bot nodejs-github-bot added libuv Issues and PRs related to the libuv dependency or the uv binding. needs-ci PRs that need a full CI run. labels Nov 4, 2026
@github-actions

github-actions Bot commented Nov 4, 2026

Copy link
Copy Markdown
Contributor

Welcome to Node.js, and thank you for your first contribution!

Before review, please take a moment to read:

Please make sure every commit is signed off. For a first pull request, GitHub Actions require collaborator approval and Jenkins CI must be started by a collaborator or triager, so an initial wait is normal.

PTY output can be truncated when a hangup arrives before all buffered
data has been read. I ran into this while using a terminal emulator
library.

Asking for a small, targeted backport of this upstream libuv fix to
read the remaining data before EOF.

Refs: libuv/libuv#5165
Signed-off-by: Alex Forster <alex@alexforster.com>
@alexforster
alexforster force-pushed the deps-backport-libuv-pty-hangup branch from 18cf537 to 6bbf8e0 Compare November 4, 2026 03:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

libuv Issues and PRs related to the libuv dependency or the uv binding. needs-ci PRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants