Skip to content

doc: maxReservedRemoteStreams value constraints - #26309

Closed
sebdeckers wants to merge 1 commit into
masterfrom
doc-max-push-default
Closed

doc: maxReservedRemoteStreams value constraints#26309
sebdeckers wants to merge 1 commit into
masterfrom
doc-max-push-default

Conversation

@sebdeckers

Copy link
Copy Markdown
Contributor

Source:
https://nghttp2.org/documentation/nghttp2_option_set_max_reserved_remote_streams.html

Checklist

/cc @nodejs/http2

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@sebdeckers sadly an error occured when I tried to trigger a build :(

@nodejs-github-botnodejs-github-bot added doc Issues and PRs related to the documentations. http2 Issues or PRs related to the http2 subsystem. labels Feb 26, 2019
@Trott

Trott commented Feb 26, 2019

Copy link
Copy Markdown
Member

@TrottTrott added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Feb 26, 2019

@mcollinamcollina 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.

LGTM

@Trott

Copy link
Copy Markdown
Member

@sebdeckers sadly an error occured when I tried to trigger a build :(

FYI, I'm pretty sure that's just because CI is locked down right now while the upcoming security release is prepared. (Mentioning it only since @sebdeckers gave it a 😕 reaction.)

@addaleax

Copy link
Copy Markdown
Member

Landed in 2a5edaf

@addaleaxaddaleax closed this Mar 2, 2019
@addaleax
addaleax deleted the doc-max-push-default branch March 2, 2019 13:13
addaleax pushed a commit that referenced this pull request Mar 2, 2019
PR-URL: #26309
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
addaleax pushed a commit that referenced this pull request Mar 2, 2019
PR-URL: #26309
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@BridgeARBridgeAR mentioned this pull request Mar 4, 2019
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.docIssues and PRs related to the documentations.http2Issues or PRs related to the http2 subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants

@sebdeckers@nodejs-github-bot@Trott@addaleax@mcollina@jasnell@lpinca@cjihrig