mirror of
https://github.com/nodejs/node.git
synced 2026-10-28 10:15:07 +08:00
Increase the default HTTP/2 stream window from 64KB (65535) to 4MB (4194304) and the default local connection window to 32MB (33554432). The default 64KB window limits throughput on high-latency connections to window_size / RTT. With a 250ms RTT, throughput is limited to 256KB/s. The new defaults improve throughput to 16MB/s (128Mbps) for the stream window and 128MB/s (1Gbps) for the connection window. Fixes: https://github.com/nodejs/node/issues/38426 Signed-off-by: Matteo Collina <hello@matteocollina.com> PR-URL: https://github.com/nodejs/node/pull/64623 Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com> Reviewed-By: Tim Perry <pimterry@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Gürgün Dayıoğlu <hey@gurgun.day> Reviewed-By: Ethan Arrowood <ethan@arrowood.dev> Reviewed-By: Aviv Keller <me@aviv.sh> Reviewed-By: Robert Nagy <ronagy@icloud.com> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
109 lines
3.5 KiB
JavaScript
109 lines
3.5 KiB
JavaScript
// Flags: --no-warnings
|
|
'use strict';
|
|
|
|
const common = require('../common');
|
|
if (!common.hasCrypto)
|
|
common.skip('missing crypto');
|
|
const assert = require('assert');
|
|
const h2 = require('http2');
|
|
|
|
const { PerformanceObserver } = require('perf_hooks');
|
|
|
|
const obs = new PerformanceObserver(common.mustCallAtLeast((items) => {
|
|
const entry = items.getEntries()[0];
|
|
assert.strictEqual(entry.entryType, 'http2');
|
|
assert.strictEqual(typeof entry.startTime, 'number');
|
|
assert.strictEqual(typeof entry.duration, 'number');
|
|
switch (entry.name) {
|
|
case 'Http2Session':
|
|
assert.strictEqual(typeof entry.detail.pingRTT, 'number');
|
|
assert.strictEqual(typeof entry.detail.streamAverageDuration, 'number');
|
|
assert.strictEqual(typeof entry.detail.streamCount, 'number');
|
|
assert.strictEqual(typeof entry.detail.framesReceived, 'number');
|
|
assert.strictEqual(typeof entry.detail.framesSent, 'number');
|
|
assert.strictEqual(typeof entry.detail.bytesWritten, 'number');
|
|
assert.strictEqual(typeof entry.detail.bytesRead, 'number');
|
|
assert.strictEqual(typeof entry.detail.maxConcurrentStreams, 'number');
|
|
switch (entry.detail.type) {
|
|
case 'server':
|
|
assert.strictEqual(entry.detail.streamCount, 1);
|
|
assert(entry.detail.framesReceived >= 3);
|
|
break;
|
|
case 'client':
|
|
assert.strictEqual(entry.detail.streamCount, 1);
|
|
// With the default window size increase (#38426), the server now
|
|
// sends a WINDOW_UPDATE frame during session setup, increasing
|
|
// the frames received by the client from 7 to 8.
|
|
assert.ok(entry.detail.framesReceived >= 7,
|
|
`expected >=7 frames but got ${entry.detail.framesReceived}`);
|
|
break;
|
|
default:
|
|
assert.fail('invalid Http2Session type');
|
|
}
|
|
break;
|
|
case 'Http2Stream':
|
|
assert.strictEqual(typeof entry.detail.timeToFirstByte, 'number');
|
|
assert.strictEqual(typeof entry.detail.timeToFirstByteSent, 'number');
|
|
assert.strictEqual(typeof entry.detail.timeToFirstHeader, 'number');
|
|
assert.strictEqual(typeof entry.detail.bytesWritten, 'number');
|
|
assert.strictEqual(typeof entry.detail.bytesRead, 'number');
|
|
break;
|
|
default:
|
|
assert.fail('invalid entry name');
|
|
}
|
|
}));
|
|
|
|
obs.observe({ type: 'http2' });
|
|
|
|
const body =
|
|
'<html><head></head><body><h1>this is some data</h2></body></html>';
|
|
|
|
const server = h2.createServer();
|
|
|
|
// We use the lower-level API here
|
|
server.on('stream', common.mustCall(onStream));
|
|
|
|
function onStream(stream, headers, flags) {
|
|
assert.strictEqual(headers[':scheme'], 'http');
|
|
assert.ok(headers[':authority']);
|
|
assert.strictEqual(headers[':method'], 'GET');
|
|
assert.strictEqual(flags, 5);
|
|
stream.respond({
|
|
'content-type': 'text/html',
|
|
':status': 200
|
|
});
|
|
stream.write(body.slice(0, 20));
|
|
stream.end(body.slice(20));
|
|
}
|
|
|
|
server.on('session', common.mustCall((session) => {
|
|
session.ping(common.mustCall());
|
|
}));
|
|
|
|
server.listen(0);
|
|
|
|
server.on('listening', common.mustCall(() => {
|
|
|
|
const client = h2.connect(`http://localhost:${server.address().port}`);
|
|
|
|
client.on('connect', common.mustCall(() => {
|
|
client.ping(common.mustCall());
|
|
}));
|
|
|
|
const req = client.request();
|
|
|
|
req.on('response', common.mustCall());
|
|
|
|
let data = '';
|
|
req.setEncoding('utf8');
|
|
req.on('data', (d) => data += d);
|
|
req.on('end', common.mustCall(() => {
|
|
assert.strictEqual(body, data);
|
|
}));
|
|
req.on('close', common.mustCall(() => {
|
|
client.close();
|
|
server.close();
|
|
}));
|
|
|
|
}));
|