Skip to content

Commit

Permalink
test: apply test-http2-max-session-memory-leak from v12.x
Browse files Browse the repository at this point in the history
Refs: #27914

Backport-PR-URL: #29123
PR-URL: #29122
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: James M Snell <[email protected]>
  • Loading branch information
addaleax authored and BethGriggs committed Aug 15, 2019
1 parent 0acbe05 commit d85e400
Showing 1 changed file with 46 additions and 0 deletions.
46 changes: 46 additions & 0 deletions test/parallel/test-http2-max-session-memory-leak.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,46 @@
'use strict';
const common = require('../common');
if (!common.hasCrypto)
common.skip('missing crypto');
const http2 = require('http2');

// Regression test for https://github.com/nodejs/node/issues/27416.
// Check that received data is accounted for correctly in the maxSessionMemory
// mechanism.

const bodyLength = 8192;
const maxSessionMemory = 1; // 1 MB
const requestCount = 1000;

const server = http2.createServer({ maxSessionMemory });
server.on('stream', (stream) => {
stream.respond();
stream.end();
});

server.listen(common.mustCall(() => {
const client = http2.connect(`http://localhost:${server.address().port}`, {
maxSessionMemory
});

function request() {
return new Promise((resolve, reject) => {
const stream = client.request({
':method': 'POST',
'content-length': bodyLength
});
stream.on('error', reject);
stream.on('response', resolve);
stream.end('a'.repeat(bodyLength));
});
}

(async () => {
for (let i = 0; i < requestCount; i++) {
await request();
}

client.close();
server.close();
})().then(common.mustCall());
}));

0 comments on commit d85e400

Please sign in to comment.