diff --git a/lib/_http_server.js b/lib/_http_server.js index a88900aaf33e03..8fa2aff627bc48 100644 --- a/lib/_http_server.js +++ b/lib/_http_server.js @@ -364,7 +364,7 @@ function Server(options, requestListener) { this.keepAliveTimeout = 5000; this.maxHeadersCount = null; this.headersTimeout = 60 * 1000; // 60 seconds - this.requestTimeout = 0; // 120 seconds + this.requestTimeout = 0; } ObjectSetPrototypeOf(Server.prototype, net.Server.prototype); ObjectSetPrototypeOf(Server, net.Server); diff --git a/lib/https.js b/lib/https.js index 4b21977fe48c3a..6aafef6cb639e1 100644 --- a/lib/https.js +++ b/lib/https.js @@ -80,7 +80,7 @@ function Server(opts, requestListener) { this.keepAliveTimeout = 5000; this.maxHeadersCount = null; this.headersTimeout = 60 * 1000; // 60 seconds - this.requestTimeout = 120 * 1000; // 120 seconds + this.requestTimeout = 0; } ObjectSetPrototypeOf(Server.prototype, tls.Server.prototype); ObjectSetPrototypeOf(Server, tls.Server); diff --git a/test/parallel/test-https-server-request-timeout.js b/test/parallel/test-https-server-request-timeout.js new file mode 100644 index 00000000000000..66a1cb9f25f82e --- /dev/null +++ b/test/parallel/test-https-server-request-timeout.js @@ -0,0 +1,21 @@ +'use strict'; + +const common = require('../common'); +if (!common.hasCrypto) + common.skip('missing crypto'); +const assert = require('assert'); +const { createServer } = require('https'); +const fixtures = require('../common/fixtures'); + +const options = { + key: fixtures.readKey('agent1-key.pem'), + cert: fixtures.readKey('agent1-cert.pem') +}; + +const server = createServer(options); + +// 0 seconds is the default +assert.strictEqual(server.requestTimeout, 0); +const requestTimeout = common.platformTimeout(1000); +server.requestTimeout = requestTimeout; +assert.strictEqual(server.requestTimeout, requestTimeout);