Skip to content

test: add Headers coverage and benchmark - #65365

Open
anonrig wants to merge 1 commit into
nodejs:mainfrom
anonrig:cursor/headers-class-performance-859a
Open

test: add Headers coverage and benchmark#65365
anonrig wants to merge 1 commit into
nodejs:mainfrom
anonrig:cursor/headers-class-performance-859a

Conversation

@anonrig

@anonrig anonrig commented Aug 18, 2026

Copy link
Copy Markdown
Member

Adds Node-side coverage for the WHATWG Headers API and a fetch/headers benchmark.

These tests are not from WPT. They cover constructor init, get/set/append/delete, set-cookie / getSetCookie(), iteration (including mutation during iteration), validation, brand checks, and util.inspect.

The implementation change lives in undici:

Refs: nodejs/undici#5699

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/performance
  • @nodejs/security-wg

@nodejs-github-bot nodejs-github-bot added dependencies Pull requests that update a dependency file. needs-ci PRs that need a full CI run. labels Aug 18, 2026
@cursor
cursor Bot force-pushed the cursor/headers-class-performance-859a branch from 3ea8120 to eb22154 Compare August 18, 2026 01:59
@anonrig anonrig changed the title deps: improve Headers class performance test: add Headers coverage and benchmark Aug 18, 2026
Add WHATWG Headers unit tests and a fetch/headers benchmark so Node
can track the API after the implementation change lands in undici.

Refs: nodejs/undici#5699
Co-authored-by: Yagiz Nizipli <yagiz@nizipli.com>
Signed-off-by: Cursor Agent <cursoragent@cursor.com>
@cursor
cursor Bot force-pushed the cursor/headers-class-performance-859a branch from eb22154 to 2edacb0 Compare August 18, 2026 02:06
@anonrig
anonrig marked this pull request as ready for review August 18, 2026 02:06

@jasnell jasnell left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

AI agents are not permitted to use the Signed-off-by

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file. needs-ci PRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants