Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

deps: update nghttp2 to 1.38.0 #27295

Closed
wants to merge 1 commit into from
Closed

Conversation

gengjiawen
Copy link
Member

Release: https://github.com/nghttp2/nghttp2/releases/tag/v1.38.0
Mainly Fix nghttp2/nghttp2#1331.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines

@nodejs-github-bot nodejs-github-bot added dont-land-on-v6.x http2 Issues or PRs related to the http2 subsystem. labels Apr 18, 2019
@gengjiawen
Copy link
Member Author

Need to add semver-minor label.

@ZYSzys ZYSzys added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. semver-minor PRs that contain new features and should be released in the next minor version. labels Apr 21, 2019
@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@ZYSzys
Copy link
Member

ZYSzys commented Apr 22, 2019

Landed in 2161690 🎉

@ZYSzys ZYSzys closed this Apr 22, 2019
ZYSzys pushed a commit that referenced this pull request Apr 22, 2019
PR-URL: #27295
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
Reviewed-By: Masashi Hirano <shisama07@gmail.com>
@addaleax addaleax removed the semver-minor PRs that contain new features and should be released in the next minor version. label Apr 23, 2019
@addaleax
Copy link
Member

@gengjiawen This doesn’t appear to be a semver-minor change – our cherry-pick only contains a semver-patch part.

@BethGriggs BethGriggs mentioned this pull request Aug 15, 2019
BethGriggs pushed a commit that referenced this pull request Aug 15, 2019
PR-URL: #27295
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
Reviewed-By: Masashi Hirano <shisama07@gmail.com>
This was referenced Aug 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
author ready PRs that have at least one approval, no pending requests for changes, and a CI started. http2 Issues or PRs related to the http2 subsystem.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Headers emitted on closed streams with no sanitization applied
7 participants