Skip to content

Commit

Permalink
http2: do not override the allowHalfOpen option
Browse files Browse the repository at this point in the history
The constructors of `tls.Server` and `Http2Server` call the super
constructors without options so the `allowHalfOpen` option is never used
regardless of its value.

PR-URL: nodejs#27623
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
  • Loading branch information
lpinca authored and danbev committed May 13, 2019
1 parent 639b859 commit 3309c85
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion lib/internal/http2/core.js
Original file line number Diff line number Diff line change
Expand Up @@ -2641,7 +2641,6 @@ function connectionListener(socket) {
function initializeOptions(options) {
assertIsObject(options, 'options');
options = { ...options };
options.allowHalfOpen = true;
assertIsObject(options.settings, 'options.settings');
options.settings = { ...options.settings };

Expand Down

0 comments on commit 3309c85

Please sign in to comment.