Skip to content

Commit

Permalink
http2: fixed linting issues
Browse files Browse the repository at this point in the history
Refs: #33636
  • Loading branch information
preyunk committed May 29, 2020
1 parent 205f38a commit 2f3782c
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions lib/internal/http2/util.js
Original file line number Diff line number Diff line change
Expand Up @@ -353,8 +353,8 @@ function updateSettingsBuffer(settings) {
if (typeof settings.maxHeaderListSize === 'number' ||
typeof settings.maxHeaderSize === 'number') {
flags |= (1 << IDX_SETTINGS_MAX_HEADER_LIST_SIZE);
if(settings.maxHeaderListSize !== undefined &&
(settings.maxHeaderListSize !== maxHeaderSize)) {
if (settings.maxHeaderListSize !== undefined &&
(settings.maxHeaderListSize !== settings.maxHeaderSize)) {
throw new ERR_HTTP2_INVALID_SETTING_VALUE.RangeError(
'maxHeaderListSize', settings.maxHeaderListSize
);
Expand Down
2 changes: 1 addition & 1 deletion test/parallel/test-http2-too-large-headers.js
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ const {
let server = http2.createServer({ settings: { maxHeaderListSize: 100 } });
server.on('stream', common.mustNotCall());

server = http2.createServer({ settings: {maxHeaderSize: 100 } });
server = http2.createServer({ settings: { maxHeaderSize: 100 } });
server.on('stream', common.mustNotCall());

server.listen(0, common.mustCall(() => {
Expand Down

0 comments on commit 2f3782c

Please sign in to comment.