Skip to content

Commit d3d662a

Browse files
hkleungaiaduh95
authored andcommitted
http2: remove no longer userful options.selectPadding
This is a followup cleanup on a previous PR. Current source code and doc explicitly states that `selectPadding` has been removed for all exports. Refs: #29144 Refs: https://nodejs.org/api/http2.html PR-URL: #58373 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Rafael Gonzaga <[email protected]> Reviewed-By: James M Snell <[email protected]>
1 parent 98b6aa0 commit d3d662a

File tree

1 file changed

+0
-3
lines changed

1 file changed

+0
-3
lines changed

lib/internal/http2/core.js

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -254,7 +254,6 @@ const kProceed = Symbol('proceed');
254254
const kProtocol = Symbol('protocol');
255255
const kRemoteSettings = Symbol('remote-settings');
256256
const kRequestAsyncResource = Symbol('requestAsyncResource');
257-
const kSelectPadding = Symbol('select-padding');
258257
const kSentHeaders = Symbol('sent-headers');
259258
const kSentTrailers = Symbol('sent-trailers');
260259
const kServer = Symbol('server');
@@ -1108,8 +1107,6 @@ function setupHandle(socket, type, options) {
11081107
const handle = new binding.Http2Session(type);
11091108
handle[kOwner] = this;
11101109

1111-
if (typeof options.selectPadding === 'function')
1112-
this[kSelectPadding] = options.selectPadding;
11131110
handle.consume(socket._handle);
11141111
handle.ongracefulclosecomplete = this[kMaybeDestroy].bind(this, null);
11151112

0 commit comments

Comments
 (0)