Skip to content

Implement the http/https outgoing message tail, header values and Agent socket hooks - #3547

Open
ctate wants to merge 12 commits into
mainfrom
ctate/implement-http-https-749b58
Open

ctate wants to merge 12 commits into
mainfrom
ctate/implement-http-https-749b58

Conversation

@ctate

@ctate ctate commented Nov 11, 2026

Copy link
Copy Markdown
Collaborator
  • Moves 36 compatibility rows: 12 not-implemented → partial (writeInformation, setHeaders, pipe(), 'drain', writableLength, IncomingMessage.signal, http/https server[Symbol.asyncDispose](), and the Agent's createConnection/keepSocketAlive/reuseSocket/maxTotalSockets) and 24 partial → supported (typed outgoing header values, the request and response header readers, validateHeaderName/validateHeaderValue, METHODS, STATUS_CODES, maxHeaderSize, socket, writableFinished, 'finish').
  • Implements Node 24's exact behavior — interim 1xx writes and their error order, setHeaders with exact-case set-cookie accumulation, a per-message AbortSignal that links only when read, the ERR_STREAM_CANNOT_PIPE throw/emit split, Writable backpressure, disposals that reject with ERR_SERVER_NOT_RUNNING, header values that keep their kind, and the Agent's dial ladder and total-socket cap.
  • Adds ten differential corpus programs covering the success and error paths, and leaves outgoingMessage.end()'s encoding/callback overloads partial with setGlobalProxyFromEnv as the one http member still absent.

@vercel

vercel Bot commented Nov 11, 2026 •

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
scriptc Ready Ready Preview, v0 Nov 12, 2026 10:58pm UTC

@vercel-internal-security-reviewer

Copy link
Copy Markdown

Security review details

Comment thread packages/runtime/src/scr_http.c Resolved
Comment thread packages/runtime/src/scr_http.c Resolved
Comment thread packages/runtime/src/scr_http.c Resolved
Comment thread packages/runtime/src/scr_http.c Resolved

This branch was successfully deployed

1 active deployment
Preview — c67e5766 Deployed Nov 12, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant