Skip to content

doc[http2]: update push and trailers event with rawHeaders param#63259

Open
samuel871211 wants to merge 1 commit into
nodejs:mainfrom
samuel871211:pushAndTrailers
Open

doc[http2]: update push and trailers event with rawHeaders param#63259
samuel871211 wants to merge 1 commit into
nodejs:mainfrom
samuel871211:pushAndTrailers

Conversation

@samuel871211
Copy link
Copy Markdown
Contributor

@samuel871211 samuel871211 commented May 12, 2026 •
edited
Loading

https://github.com/nodejs/node/blob/main/lib/internal/http2/core.js#L357

function onSessionHeaders
    if (reqAsync)
      reqAsync.runInAsyncScope(process.nextTick, null, emit, stream, event, obj, flags, headers);
    else
      process.nextTick(emit, stream, event, obj, flags, headers);

@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

nodejs-github-bot commented May 12, 2026

Review requested:

  • @nodejs/http
  • @nodejs/http2
  • @nodejs/net

@nodejs-github-bot nodejs-github-bot added doc Issues and PRs related to the documentations. http2 Issues or PRs related to the http2 subsystem. labels May 12, 2026
@samuel871211 samuel871211 force-pushed the pushAndTrailers branch 2 times, most recently from 8505d4d to c1bb7b4 Compare May 12, 2026 01:13
@samuel871211
http2: update push and trailers event with rawHeaders param
a76836b 
Signed-off-by: Yu-Sheng Chen <samuel871211@gmail.com>
pimterry
pimterry approved these changes May 12, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@pimterry pimterry left a comment

Choose a reason for hiding this comment

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

Nice addition, thanks for spotting & fixing this! 👍

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

Reviewers

@pimterry pimterry pimterry approved these changes

Assignees

No one assigned

Labels

doc Issues and PRs related to the documentations. http2 Issues or PRs related to the http2 subsystem.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@samuel871211 @nodejs-github-bot @pimterry