Skip to content

Conversation

@lpinca
Copy link
Member

Make http2.createServer() support net.Server options.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines

Make `http2.createServer()` support `net.Server` options.
@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-botnodejs-github-bot added the http2 Issues or PRs related to the http2 subsystem. label May 20, 2019
@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@trivikrtrivikr added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label May 22, 2019
@danbev
Copy link
Contributor

Landed in 165916b.

@danbevdanbev closed this May 23, 2019
danbev pushed a commit that referenced this pull request May 23, 2019
Make `http2.createServer()` support `net.Server` options. PR-URL: #27782 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Yongsheng Zhang <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]>
@lpincalpinca deleted the pass/options-to-parent-ctor branch May 23, 2019 05:48
targos pushed a commit that referenced this pull request May 28, 2019
Make `http2.createServer()` support `net.Server` options. PR-URL: #27782 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Yongsheng Zhang <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]>
@targostargos mentioned this pull request Jun 3, 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.http2Issues or PRs related to the http2 subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants

@lpinca@nodejs-github-bot@danbev@Trott@addaleax@cjihrig@BridgeAR@trivikr@ZYSzys