diff options
-rw-r--r-- | lib/_http_server.js | 2 | ||||
-rw-r--r-- | lib/https.js | 2 | ||||
-rw-r--r-- | test/parallel/test-https-server-request-timeout.js | 21 |
3 files changed, 23 insertions, 2 deletions
diff --git a/lib/_http_server.js b/lib/_http_server.js index 21eb68d463..4923c0b66a 100644 --- a/lib/_http_server.js +++ b/lib/_http_server.js @@ -373,7 +373,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 4b21977fe4..6aafef6cb6 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 0000000000..66a1cb9f25 --- /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); |