benchmark: fixing http_server_for_chunky_client.js

PR-URL: #33271 Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]>
nodejs · Jun 30, 2020 · c0534f7 · c0534f7
1 parent 81a875f
commit c0534f7
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/benchmark/http/http_server_for_chunky_client.js b/benchmark/http/http_server_for_chunky_client.js
@@ -10,7 +10,9 @@ process.env.PIPE_NAME = PIPE;
 
 tmpdir.refresh();
 
-const server = http.createServer((req, res) => {
+// For Node.js versions below v13.3.0 this benchmark will require
+// the flag --max-http-header-size 64000 in order to work properly
+const server = http.createServer({ maxHeaderSize: 64000 }, (req, res) => {
   const headers = {
     'content-type': 'text/plain',
     'content-length': '2'
@@ -28,7 +30,11 @@ const child = fork(
   `${__dirname}/_chunky_http_client.js`,
   process.argv.slice(2)
 );
-child.on('message', common.sendResult);
+child.on('message', (data) => {
+  if (data.type === 'report') {
+    common.sendResult(data);
+  }
+});
 child.on('close', (code) => {
   server.close();
   assert.strictEqual(code, 0);