From 4b7f7c1dcfe437abc87ec22892bdc4eaf54df3c6 Mon Sep 17 00:00:00 2001 From: Deokjin Kim Date: Wed, 8 Mar 2023 23:43:51 +0900 Subject: [PATCH] net: fix setting of value in 'setDefaultAutoSelectFamilyAttemptTimeout' Document describes that the value have to be 10 if passed value to `setDefaultAutoSelectFamilyAttemptTimeout` is less than 10. So need to use 10 for 'if' statement and fix typo in document. Refs: https://github.com/nodejs/node/blob/main/doc/api/net.md#netsetdefaultautoselectfamilyattempttimeoutvalue --- doc/api/net.md | 2 +- lib/net.js | 2 +- ...ocket-connect-invalid-autoselectfamilyattempttimeout.js | 7 +++++++ 3 files changed, 9 insertions(+), 2 deletions(-) diff --git a/doc/api/net.md b/doc/api/net.md index b752129dd55fba..2b67e85a32d9e2 100644 --- a/doc/api/net.md +++ b/doc/api/net.md @@ -1661,7 +1661,7 @@ added: REPLACEME Sets the default value of the `autoSelectFamilyAttemptTimeout` option of [`socket.connect(options)`][]. * `value` {number} The new default value, which must be a positive number. If the number is less than `10`, - the value `10` is used insted The initial default value is `250`. + the value `10` is used instead. The initial default value is `250`. ## `net.isIP(input)` diff --git a/lib/net.js b/lib/net.js index 7ac8a0bdc649c5..654190551e46d9 100644 --- a/lib/net.js +++ b/lib/net.js @@ -247,7 +247,7 @@ function getDefaultAutoSelectFamilyAttemptTimeout() { function setDefaultAutoSelectFamilyAttemptTimeout(value) { validateInt32(value, 'value', 1); - if (value < 1) { + if (value < 10) { value = 10; } diff --git a/test/parallel/test-net-socket-connect-invalid-autoselectfamilyattempttimeout.js b/test/parallel/test-net-socket-connect-invalid-autoselectfamilyattempttimeout.js index adec8200f697d2..0fc813781c95ec 100644 --- a/test/parallel/test-net-socket-connect-invalid-autoselectfamilyattempttimeout.js +++ b/test/parallel/test-net-socket-connect-invalid-autoselectfamilyattempttimeout.js @@ -18,3 +18,10 @@ for (const autoSelectFamilyAttemptTimeout of [-10, 0]) { net.setDefaultAutoSelectFamilyAttemptTimeout(autoSelectFamilyAttemptTimeout); }, { code: 'ERR_OUT_OF_RANGE' }); } + +// Check the default value of autoSelectFamilyAttemptTimeout is 10 +// if passed number is less than 10 +for (const autoSelectFamilyAttemptTimeout of [1, 9]) { + net.setDefaultAutoSelectFamilyAttemptTimeout(autoSelectFamilyAttemptTimeout); + assert.strictEqual(net.getDefaultAutoSelectFamilyAttemptTimeout(), 10); +}