From 6ba9471f6fcaaf377f22621c4d193d30e3877566 Mon Sep 17 00:00:00 2001 From: ZYSzys Date: Tue, 1 Oct 2019 23:06:44 +0800 Subject: http2: use the latest settings Fixes: https://github.com/nodejs/node/issues/29764 PR-URL: https://github.com/nodejs/node/pull/29780 Reviewed-By: James M Snell Reviewed-By: Luigi Pinca Reviewed-By: Anna Henningsen Reviewed-By: Colin Ihrig Reviewed-By: Beth Griggs Reviewed-By: Anto Aravinth Reviewed-By: Ruben Bridgewater --- test/parallel/test-http2-session-settings.js | 15 ++++++++++++++- 1 file changed, 14 insertions(+), 1 deletion(-) (limited to 'test') diff --git a/test/parallel/test-http2-session-settings.js b/test/parallel/test-http2-session-settings.js index 46ba1753e2..35581d5f71 100644 --- a/test/parallel/test-http2-session-settings.js +++ b/test/parallel/test-http2-session-settings.js @@ -38,6 +38,12 @@ server.on( }) ); +server.on('session', (session) => { + session.settings({ + maxConcurrentStreams: 2 + }); +}); + server.listen( 0, common.mustCall(() => { @@ -57,11 +63,18 @@ server.listen( assert.strictEqual(settings.maxFrameSize, 16384); }, 2) ); + + let calledOnce = false; client.on( 'remoteSettings', common.mustCall((settings) => { assert(settings); - }) + assert.strictEqual( + settings.maxConcurrentStreams, + calledOnce ? 2 : (2 ** 32) - 1 + ); + calledOnce = true; + }, 2) ); const headers = { ':path': '/' }; -- cgit v1.2.3