Skip to content

Commit 661fd6b

Browse files
puzpuzpuzpuzpuzpuz
committed
doc,test: add server.timeout property to http2 public API
Both http and https modules have server.timeout property in public API. This commit adds documentation section and test for server.timeout in http2 module, so it becomes consistent with http and https. Also improves description of callback argument in documentation for server.setTimeout().
1 parent 8a14884 commit 661fd6b

2 files changed

Lines changed: 45 additions & 3 deletions

File tree

‎doc/api/http2.md‎

Lines changed: 40 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1797,9 +1797,28 @@ on the `Http2Server` after `msecs` milliseconds.
17971797

17981798
The given callback is registered as a listener on the `'timeout'` event.
17991799

1800-
In case of no callback function were assigned, a new `ERR_INVALID_CALLBACK`
1800+
In case if `callback` is not a function, a new `ERR_INVALID_CALLBACK`
18011801
error will be thrown.
18021802

1803+
#### `server.timeout`
1804+
<!-- YAML
1805+
added: v8.4.0
1806+
changes:
1807+
- version: v13.0.0
1808+
pr-url: https://github.com/nodejs/node/pull/27558
1809+
description: The default timeout changed from 120s to 0 (no timeout).
1810+
-->
1811+
1812+
* {number} Timeout in milliseconds. **Default:** 0 (no timeout)
1813+
1814+
The number of milliseconds of inactivity before a socket is presumed
1815+
to have timed out.
1816+
1817+
A value of `0` will disable the timeout behavior on incoming connections.
1818+
1819+
The socket timeout logic is set up on connection, so changing this
1820+
value only affects new connections to the server, not any existing connections.
1821+
18031822
### Class: `Http2SecureServer`
18041823
<!-- YAML
18051824
added: v8.4.0
@@ -1943,9 +1962,28 @@ on the `Http2SecureServer` after `msecs` milliseconds.
19431962

19441963
The given callback is registered as a listener on the `'timeout'` event.
19451964

1946-
In case of no callback function were assigned, a new `ERR_INVALID_CALLBACK`
1965+
In case if `callback` is not a function, a new `ERR_INVALID_CALLBACK`
19471966
error will be thrown.
19481967

1968+
#### `server.timeout`
1969+
<!-- YAML
1970+
added: v8.4.0
1971+
changes:
1972+
- version: v13.0.0
1973+
pr-url: https://github.com/nodejs/node/pull/27558
1974+
description: The default timeout changed from 120s to 0 (no timeout).
1975+
-->
1976+
1977+
* {number} Timeout in milliseconds. **Default:** 0 (no timeout)
1978+
1979+
The number of milliseconds of inactivity before a socket is presumed
1980+
to have timed out.
1981+
1982+
A value of `0` will disable the timeout behavior on incoming connections.
1983+
1984+
The socket timeout logic is set up on connection, so changing this
1985+
value only affects new connections to the server, not any existing connections.
1986+
19491987
### `http2.createServer(options[, onRequestHandler])`
19501988
<!-- YAML
19511989
added: v8.4.0

‎test/parallel/test-http2-server-timeout.js‎

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,10 +3,14 @@
33
const common = require('../common');
44
if (!common.hasCrypto)
55
common.skip('missing crypto');
6+
const assert = require('assert');
67
const http2 = require('http2');
78

89
const server = http2.createServer();
9-
server.setTimeout(common.platformTimeout(50));
10+
server.setTimeout(42000);
11+
assert.strictEqual(server.timeout, 42000);
12+
13+
server.timeout = common.platformTimeout(50);
1014

1115
const onServerTimeout = common.mustCall((session) => {
1216
session.close();

0 commit comments

Comments
 (0)