From 55321bafd1600af3d35475756a43ef66851413ef Mon Sep 17 00:00:00 2001 From: Paolo Insogna Date: Thu, 23 Feb 2023 10:55:04 +0100 Subject: [PATCH] net: enable autoSelectFamily by default Co-authored-by: Antoine du Hamel PR-URL: https://github.com/nodejs/node/pull/46790 Reviewed-By: Matteo Collina Reviewed-By: Robert Nagy Reviewed-By: James M Snell Reviewed-By: Antoine du Hamel Reviewed-By: Rich Trott Reviewed-By: Rafael Gonzaga --- doc/api/cli.md | 13 +++- doc/api/errors.md | 7 ++ doc/api/net.md | 19 +++-- lib/internal/errors.js | 2 + lib/internal/net.js | 2 +- lib/net.js | 70 ++++++++++++++----- src/node_options.cc | 11 +-- src/node_options.h | 2 +- test/common/index.js | 10 +++ .../test-tls-autoselectfamily-servername.js | 4 -- test/parallel/test-http-autoselectfamily.js | 4 -- .../test-http2-ping-settings-heapdump.js | 11 +-- test/parallel/test-https-autoselectfamily.js | 4 -- ...net-autoselectfamily-commandline-option.js | 34 ++++----- ...s => test-net-autoselectfamily-default.js} | 6 +- .../test-net-autoselectfamily-ipv4first.js | 5 +- test/parallel/test-net-autoselectfamily.js | 4 +- .../parallel/test-tls-connect-hints-option.js | 2 +- test/sequential/test-http-econnrefused.js | 6 +- .../test-net-better-error-messages-port.js | 11 +-- .../test-net-connect-econnrefused.js | 7 +- test/sequential/test-net-reconnect-error.js | 7 +- 22 files changed, 147 insertions(+), 94 deletions(-) rename test/parallel/{test-net-autoselectfamilydefault.js => test-net-autoselectfamily-default.js} (94%) diff --git a/doc/api/cli.md b/doc/api/cli.md index cda8232ac91d9f..a866f73de17ac0 100644 --- a/doc/api/cli.md +++ b/doc/api/cli.md @@ -462,14 +462,20 @@ added: v6.0.0 Enable FIPS-compliant crypto at startup. (Requires Node.js to be built against FIPS-compatible OpenSSL.) -### `--enable-network-family-autoselection` +### `--no-network-family-autoselection` -Enables the family autoselection algorithm unless connection options explicitly -disables it. +Disables the family autoselection algorithm unless connection options explicitly +enables it. ### `--enable-source-maps` @@ -2125,6 +2131,7 @@ Node.js options that are allowed are: * `--no-extra-info-on-fatal-exception` * `--no-force-async-hooks-checks` * `--no-global-search-paths` +* `--no-network-family-autoselection` * `--no-warnings` * `--node-memory-debug` * `--openssl-config` diff --git a/doc/api/errors.md b/doc/api/errors.md index a0ab195ef49bd6..2336274e7000f3 100644 --- a/doc/api/errors.md +++ b/doc/api/errors.md @@ -2593,6 +2593,13 @@ An attempt was made to operate on an already closed socket. When calling [`net.Socket.write()`][] on a connecting socket and the socket was closed before the connection was established. + + +### `ERR_SOCKET_CONNECTION_TIMEOUT` + +The socket was unable to connect to any address returned by the DNS within the +allowed timeout when using the family autoselection algorithm. + ### `ERR_SOCKET_DGRAM_IS_CONNECTED` diff --git a/doc/api/net.md b/doc/api/net.md index ba0591baea077a..1b44aeedc75ca5 100644 --- a/doc/api/net.md +++ b/doc/api/net.md @@ -870,6 +870,12 @@ behavior. Gets the current default value of the `autoSelectFamily` option of [`socket.connect(options)`][]. +The initial default value is `true`, unless the command line option +`--no-network-family-autoselection` is provided. * Returns: {boolean} The current default value of the `autoSelectFamily` option. @@ -1649,6 +1657,7 @@ added: v19.8.0 --> Gets the current default value of the `autoSelectFamilyAttemptTimeout` option of [`socket.connect(options)`][]. +The initial default value is `250`. * Returns: {number} The current default value of the `autoSelectFamilyAttemptTimeout` option. @@ -1747,8 +1756,8 @@ net.isIPv6('fhqwhgads'); // returns false [`net.createConnection(path)`]: #netcreateconnectionpath-connectlistener [`net.createConnection(port, host)`]: #netcreateconnectionport-host-connectlistener [`net.createServer()`]: #netcreateserveroptions-connectionlistener -[`net.setDefaultAutoSelectFamily(value)`]: #netsetdefaultautoselectfamilyvalue -[`net.setDefaultAutoSelectFamilyAttemptTimeout(value)`]: #netsetdefaultautoselectfamilyattempttimeoutvalue +[`net.getDefaultAutoSelectFamily()`]: #netgetdefaultautoselectfamily +[`net.getDefaultAutoSelectFamilyAttemptTimeout()`]: #netgetdefaultautoselectfamilyattempttimeout [`new net.Socket(options)`]: #new-netsocketoptions [`readable.setEncoding()`]: stream.md#readablesetencodingencoding [`server.close()`]: #serverclosecallback diff --git a/lib/internal/errors.js b/lib/internal/errors.js index 742d723155e7fe..aa4a149b189699 100644 --- a/lib/internal/errors.js +++ b/lib/internal/errors.js @@ -1567,6 +1567,8 @@ E('ERR_SOCKET_CLOSED', 'Socket is closed', Error); E('ERR_SOCKET_CLOSED_BEFORE_CONNECTION', 'Socket closed before the connection was established', Error); +E('ERR_SOCKET_CONNECTION_TIMEOUT', + 'Socket connection timeout', Error); E('ERR_SOCKET_DGRAM_IS_CONNECTED', 'Already connected', Error); E('ERR_SOCKET_DGRAM_NOT_CONNECTED', 'Not connected', Error); E('ERR_SOCKET_DGRAM_NOT_RUNNING', 'Not running', Error); diff --git a/lib/internal/net.js b/lib/internal/net.js index 32825d4a72ac65..bcbaaa94989e28 100644 --- a/lib/internal/net.js +++ b/lib/internal/net.js @@ -67,7 +67,7 @@ function makeSyncWrite(fd) { } module.exports = { - kReinitializeHandle: Symbol('reinitializeHandle'), + kReinitializeHandle: Symbol('kReinitializeHandle'), isIP, isIPv4, isIPv6, diff --git a/lib/net.js b/lib/net.js index 654190551e46d9..e29b62c8127648 100644 --- a/lib/net.js +++ b/lib/net.js @@ -23,6 +23,7 @@ const { ArrayIsArray, + ArrayPrototypeIncludes, ArrayPrototypeIndexOf, ArrayPrototypePush, Boolean, @@ -97,6 +98,7 @@ const { ERR_INVALID_HANDLE_TYPE, ERR_SERVER_ALREADY_LISTEN, ERR_SERVER_NOT_RUNNING, + ERR_SOCKET_CONNECTION_TIMEOUT, ERR_SOCKET_CLOSED, ERR_SOCKET_CLOSED_BEFORE_CONNECTION, ERR_MISSING_ARGS, @@ -127,7 +129,7 @@ let cluster; let dns; let BlockList; let SocketAddress; -let autoSelectFamilyDefault = getOptionValue('--enable-network-family-autoselection'); +let autoSelectFamilyDefault = getOptionValue('--network-family-autoselection'); let autoSelectFamilyAttemptTimeoutDefault = 250; const { clearTimeout, setTimeout } = require('timers'); @@ -1092,6 +1094,11 @@ function internalConnectMultiple(context, canceled) { // All connections have been tried without success, destroy with error if (canceled || context.current === context.addresses.length) { + if (context.errors.length === 0) { + self.destroy(new ERR_SOCKET_CONNECTION_TIMEOUT()); + return; + } + self.destroy(aggregateErrors(context.errors)); return; } @@ -1322,6 +1329,7 @@ function lookupAndConnect(self, options) { options, dnsopts, port, + localAddress, localPort, autoSelectFamilyAttemptTimeout, ); @@ -1364,7 +1372,9 @@ function lookupAndConnect(self, options) { }); } -function lookupAndConnectMultiple(self, async_id_symbol, lookup, host, options, dnsopts, port, localPort, timeout) { +function lookupAndConnectMultiple( + self, async_id_symbol, lookup, host, options, dnsopts, port, localAddress, localPort, timeout, +) { defaultTriggerAsyncIdScope(self[async_id_symbol], function emitLookup() { lookup(host, dnsopts, function emitLookup(err, addresses) { // It's possible we were destroyed while looking this up. @@ -1385,6 +1395,7 @@ function lookupAndConnectMultiple(self, async_id_symbol, lookup, host, options, // Filter addresses by only keeping the one which are either IPv4 or IPV6. // The first valid address determines which group has preference on the // alternate family sorting which happens later. + const validAddresses = [[], []]; const validIps = [[], []]; let destinations; for (let i = 0, l = addresses.length; i < l; i++) { @@ -1397,12 +1408,19 @@ function lookupAndConnectMultiple(self, async_id_symbol, lookup, host, options, destinations = addressType === 6 ? { 6: 0, 4: 1 } : { 4: 0, 6: 1 }; } - ArrayPrototypePush(validIps[destinations[addressType]], address); + const destination = destinations[addressType]; + + // Only try an address once + if (!ArrayPrototypeIncludes(validIps[destination], ip)) { + ArrayPrototypePush(validAddresses[destination], address); + ArrayPrototypePush(validIps[destination], ip); + } } } + // When no AAAA or A records are available, fail on the first one - if (!validIps[0].length && !validIps[1].length) { + if (!validAddresses[0].length && !validAddresses[1].length) { const { address: firstIp, family: firstAddressType } = addresses[0]; if (!isIP(firstIp)) { @@ -1420,16 +1438,36 @@ function lookupAndConnectMultiple(self, async_id_symbol, lookup, host, options, // Sort addresses alternating families const toAttempt = []; - for (let i = 0, l = MathMax(validIps[0].length, validIps[1].length); i < l; i++) { - if (i in validIps[0]) { - ArrayPrototypePush(toAttempt, validIps[0][i]); + for (let i = 0, l = MathMax(validAddresses[0].length, validAddresses[1].length); i < l; i++) { + if (i in validAddresses[0]) { + ArrayPrototypePush(toAttempt, validAddresses[0][i]); } - if (i in validIps[1]) { - ArrayPrototypePush(toAttempt, validIps[1][i]); + if (i in validAddresses[1]) { + ArrayPrototypePush(toAttempt, validAddresses[1][i]); } } + if (toAttempt.length === 1) { + debug('connect/multiple: only one address found, switching back to single connection'); + const { address: ip, family: addressType } = toAttempt[0]; + + self._unrefTimer(); + defaultTriggerAsyncIdScope( + self[async_id_symbol], + internalConnect, + self, + ip, + port, + addressType, + localAddress, + localPort, + ); + + return; + } + self.autoSelectFamilyAttemptedAddresses = []; + debug('connect/multiple: will try the following addresses', toAttempt); const context = { socket: self, @@ -1543,6 +1581,13 @@ function afterConnect(status, handle, req, readable, writable) { } function afterConnectMultiple(context, status, handle, req, readable, writable) { + // One of the connection has completed and correctly dispatched but after timeout, ignore this one + if (context[kTimeoutTriggered]) { + debug('connect/multiple: ignoring successful but timedout connection to %s:%s', req.address, req.port); + handle.close(); + return; + } + const self = context.socket; // Make sure another connection is not spawned @@ -1571,13 +1616,6 @@ function afterConnectMultiple(context, status, handle, req, readable, writable) return; } - // One of the connection has completed and correctly dispatched but after timeout, ignore this one - if (context[kTimeoutTriggered]) { - debug('connect/multiple: ignoring successful but timedout connection to %s:%s', req.address, req.port); - handle.close(); - return; - } - if (context.current > 1 && self[kReinitializeHandle]) { self[kReinitializeHandle](handle); handle = self._handle; diff --git a/src/node_options.cc b/src/node_options.cc index 2a9d4deafa5dd9..73246ffb70cf9e 100644 --- a/src/node_options.cc +++ b/src/node_options.cc @@ -358,10 +358,13 @@ EnvironmentOptionsParser::EnvironmentOptionsParser() { "returned)", &EnvironmentOptions::dns_result_order, kAllowedInEnvvar); - AddOption("--enable-network-family-autoselection", - "Enable network address family autodetection algorithm", - &EnvironmentOptions::enable_network_family_autoselection, - kAllowedInEnvvar); + AddOption("--network-family-autoselection", + "Disable network address family autodetection algorithm", + &EnvironmentOptions::network_family_autoselection, + kAllowedInEnvvar, + true); + AddAlias("--enable-network-family-autoselection", + "--network-family-autoselection"); AddOption("--enable-source-maps", "Source Map V3 support for stack traces", &EnvironmentOptions::enable_source_maps, diff --git a/src/node_options.h b/src/node_options.h index 543ff419db82df..f907bd9eea0030 100644 --- a/src/node_options.h +++ b/src/node_options.h @@ -132,7 +132,7 @@ class EnvironmentOptions : public Options { bool frozen_intrinsics = false; int64_t heap_snapshot_near_heap_limit = 0; std::string heap_snapshot_signal; - bool enable_network_family_autoselection = false; + bool network_family_autoselection = true; uint64_t max_http_header_size = 16 * 1024; bool deprecation = true; bool force_async_hooks_checks = true; diff --git a/test/common/index.js b/test/common/index.js index eff02f3028da27..51a5acd32ca38b 100644 --- a/test/common/index.js +++ b/test/common/index.js @@ -26,6 +26,7 @@ const process = global.process; // Some tests tamper with the process global. const assert = require('assert'); const { exec, execSync, spawn, spawnSync } = require('child_process'); const fs = require('fs'); +const net = require('net'); // Do not require 'os' until needed so that test-os-checked-function can // monkey patch it. If 'os' is required here, that test will fail. const path = require('path'); @@ -137,6 +138,14 @@ const isPi = (() => { const isDumbTerminal = process.env.TERM === 'dumb'; +// When using high concurrency or in the CI we need much more time for each connection attempt +const defaultAutoSelectFamilyAttemptTimeout = platformTimeout(2500); +// Since this is also used by tools outside of the test suite, +// make sure setDefaultAutoSelectFamilyAttemptTimeout +if (typeof net.setDefaultAutoSelectFamilyAttemptTimeout === 'function') { + net.setDefaultAutoSelectFamilyAttemptTimeout(platformTimeout(defaultAutoSelectFamilyAttemptTimeout)); +} + const buildType = process.config.target_defaults ? process.config.target_defaults.default_configuration : 'Release'; @@ -886,6 +895,7 @@ const common = { canCreateSymLink, childShouldThrowAndAbort, createZeroFilledFile, + defaultAutoSelectFamilyAttemptTimeout, expectsError, expectWarning, gcUntil, diff --git a/test/internet/test-tls-autoselectfamily-servername.js b/test/internet/test-tls-autoselectfamily-servername.js index ae53875b677821..26f51ae6024d5f 100644 --- a/test/internet/test-tls-autoselectfamily-servername.js +++ b/test/internet/test-tls-autoselectfamily-servername.js @@ -7,12 +7,8 @@ if (!common.hasCrypto) { common.skip('missing crypto'); } -const { setDefaultAutoSelectFamilyAttemptTimeout } = require('net'); const { connect } = require('tls'); -// Some of the windows machines in the CI need more time to establish connection -setDefaultAutoSelectFamilyAttemptTimeout(common.platformTimeout(common.isWindows ? 1500 : 250)); - // Test that TLS connecting works without autoSelectFamily { const socket = connect({ diff --git a/test/parallel/test-http-autoselectfamily.js b/test/parallel/test-http-autoselectfamily.js index 6749d960f9122a..b98aacea7fc6b4 100644 --- a/test/parallel/test-http-autoselectfamily.js +++ b/test/parallel/test-http-autoselectfamily.js @@ -7,13 +7,9 @@ const assert = require('assert'); const dgram = require('dgram'); const { Resolver } = require('dns'); const { request, createServer } = require('http'); -const { setDefaultAutoSelectFamilyAttemptTimeout } = require('net'); // Test that happy eyeballs algorithm is properly implemented when using HTTP. -// Some of the windows machines in the CI need more time to establish connection -setDefaultAutoSelectFamilyAttemptTimeout(common.platformTimeout(common.isWindows ? 1500 : 250)); - function _lookup(resolver, hostname, options, cb) { resolver.resolve(hostname, 'ANY', (err, replies) => { assert.notStrictEqual(options.family, 4); diff --git a/test/parallel/test-http2-ping-settings-heapdump.js b/test/parallel/test-http2-ping-settings-heapdump.js index 775110c098607b..6023b5f6b865f2 100644 --- a/test/parallel/test-http2-ping-settings-heapdump.js +++ b/test/parallel/test-http2-ping-settings-heapdump.js @@ -11,14 +11,6 @@ const v8 = require('v8'); // after it is destroyed, either because they are detached from it or have been // destroyed themselves. -// We use an higher autoSelectFamilyAttemptTimeout in this test as the v8.getHeapSnapshot().resume() -// will slow the connection flow and we don't want the second connection attempt to start. -let autoSelectFamilyAttemptTimeout = common.platformTimeout(1000); -if (common.isWindows) { - // Some of the windows machines in the CI need more time to establish connection - autoSelectFamilyAttemptTimeout = common.platformTimeout(10000); -} - for (const variant of ['ping', 'settings']) { const server = http2.createServer(); server.on('session', common.mustCall((session) => { @@ -38,8 +30,7 @@ for (const variant of ['ping', 'settings']) { })); server.listen(0, common.mustCall(() => { - const client = http2.connect(`http://localhost:${server.address().port}`, { autoSelectFamilyAttemptTimeout }, - common.mustCall()); + const client = http2.connect(`http://localhost:${server.address().port}`, common.mustCall()); client.on('error', (err) => { // We destroy the session so it's possible to get ECONNRESET here. if (err.code !== 'ECONNRESET') diff --git a/test/parallel/test-https-autoselectfamily.js b/test/parallel/test-https-autoselectfamily.js index e9805939b5d163..cf7cc450105e7f 100644 --- a/test/parallel/test-https-autoselectfamily.js +++ b/test/parallel/test-https-autoselectfamily.js @@ -13,7 +13,6 @@ const assert = require('assert'); const dgram = require('dgram'); const { Resolver } = require('dns'); const { request, createServer } = require('https'); -const { setDefaultAutoSelectFamilyAttemptTimeout } = require('net'); if (!common.hasCrypto) common.skip('missing crypto'); @@ -25,9 +24,6 @@ const options = { // Test that happy eyeballs algorithm is properly implemented when using HTTP. -// Some of the windows machines in the CI need more time to establish connection -setDefaultAutoSelectFamilyAttemptTimeout(common.platformTimeout(common.isWindows ? 1500 : 250)); - function _lookup(resolver, hostname, options, cb) { resolver.resolve(hostname, 'ANY', (err, replies) => { assert.notStrictEqual(options.family, 4); diff --git a/test/parallel/test-net-autoselectfamily-commandline-option.js b/test/parallel/test-net-autoselectfamily-commandline-option.js index 555423374187b9..5687d31b2eec42 100644 --- a/test/parallel/test-net-autoselectfamily-commandline-option.js +++ b/test/parallel/test-net-autoselectfamily-commandline-option.js @@ -1,6 +1,6 @@ 'use strict'; -// Flags: --enable-network-family-autoselection +// Flags: --no-network-family-autoselection const common = require('../common'); const { parseDNSPacket, writeDNSPacket } = require('../common/dns'); @@ -8,13 +8,10 @@ const { parseDNSPacket, writeDNSPacket } = require('../common/dns'); const assert = require('assert'); const dgram = require('dgram'); const { Resolver } = require('dns'); -const { createConnection, createServer, setDefaultAutoSelectFamilyAttemptTimeout } = require('net'); +const { createConnection, createServer } = require('net'); // Test that happy eyeballs algorithm can be enable from command line. -// Some of the windows machines in the CI need more time to establish connection -setDefaultAutoSelectFamilyAttemptTimeout(common.platformTimeout(common.isWindows ? 1500 : 250)); - function _lookup(resolver, hostname, options, cb) { resolver.resolve(hostname, 'ANY', (err, replies) => { assert.notStrictEqual(options.family, 4); @@ -62,7 +59,7 @@ function createDnsServer(ipv6Addr, ipv4Addr, cb) { }); } -// Test that IPV4 is reached if IPV6 is not reachable +// Test that IPV4 is NOT reached if IPV6 is not reachable and the option has been disabled via command line { createDnsServer('::1', '127.0.0.1', common.mustCall(function({ dnsServer, lookup }) { const ipv4Server = createServer((socket) => { @@ -77,28 +74,25 @@ function createDnsServer(ipv6Addr, ipv4Addr, cb) { const connection = createConnection({ host: 'example.org', - port: port, + port, lookup, }); - let response = ''; - connection.setEncoding('utf-8'); + connection.on('ready', common.mustNotCall()); + connection.on('error', common.mustCall((error) => { + assert.strictEqual(connection.autoSelectFamilyAttemptedAddresses, undefined); - connection.on('ready', common.mustCall(() => { - assert.deepStrictEqual(connection.autoSelectFamilyAttemptedAddresses, [`::1:${port}`, `127.0.0.1:${port}`]); - })); + if (common.hasIPv6) { + assert.strictEqual(error.code, 'ECONNREFUSED'); + assert.strictEqual(error.message, `connect ECONNREFUSED ::1:${port}`); + } else { + assert.strictEqual(error.code, 'EADDRNOTAVAIL'); + assert.strictEqual(error.message, `connect EADDRNOTAVAIL ::1:${port} - Local (:::0)`); + } - connection.on('data', (chunk) => { - response += chunk; - }); - - connection.on('end', common.mustCall(() => { - assert.strictEqual(response, 'response-ipv4'); ipv4Server.close(); dnsServer.close(); })); - - connection.write('request'); })); })); } diff --git a/test/parallel/test-net-autoselectfamilydefault.js b/test/parallel/test-net-autoselectfamily-default.js similarity index 94% rename from test/parallel/test-net-autoselectfamilydefault.js rename to test/parallel/test-net-autoselectfamily-default.js index be110a836a057b..2c87bf97dfa87e 100644 --- a/test/parallel/test-net-autoselectfamilydefault.js +++ b/test/parallel/test-net-autoselectfamily-default.js @@ -10,11 +10,7 @@ const { createConnection, createServer, setDefaultAutoSelectFamily } = require(' // Test that the default for happy eyeballs algorithm is properly respected. -let autoSelectFamilyAttemptTimeout = common.platformTimeout(250); -if (common.isWindows) { - // Some of the windows machines in the CI need more time to establish connection - autoSelectFamilyAttemptTimeout = common.platformTimeout(1500); -} +const autoSelectFamilyAttemptTimeout = common.defaultAutoSelectFamilyAttemptTimeout; function _lookup(resolver, hostname, options, cb) { resolver.resolve(hostname, 'ANY', (err, replies) => { diff --git a/test/parallel/test-net-autoselectfamily-ipv4first.js b/test/parallel/test-net-autoselectfamily-ipv4first.js index 794f7464a80639..7c2e10213c4d28 100644 --- a/test/parallel/test-net-autoselectfamily-ipv4first.js +++ b/test/parallel/test-net-autoselectfamily-ipv4first.js @@ -6,13 +6,10 @@ const { parseDNSPacket, writeDNSPacket } = require('../common/dns'); const assert = require('assert'); const dgram = require('dgram'); const { Resolver } = require('dns'); -const { createConnection, createServer, setDefaultAutoSelectFamilyAttemptTimeout } = require('net'); +const { createConnection, createServer } = require('net'); // Test that happy eyeballs algorithm is properly implemented when a A record is returned first. -// Some of the windows machines in the CI need more time to establish connection -setDefaultAutoSelectFamilyAttemptTimeout(common.platformTimeout(common.isWindows ? 1500 : 250)); - function _lookup(resolver, hostname, options, cb) { resolver.resolve(hostname, 'ANY', (err, replies) => { assert.notStrictEqual(options.family, 4); diff --git a/test/parallel/test-net-autoselectfamily.js b/test/parallel/test-net-autoselectfamily.js index 8deff524203316..d95819000c146c 100644 --- a/test/parallel/test-net-autoselectfamily.js +++ b/test/parallel/test-net-autoselectfamily.js @@ -13,8 +13,8 @@ const { createConnection, createServer } = require('net'); // Purposely not using setDefaultAutoSelectFamilyAttemptTimeout here to test the // parameter is correctly used in options. // -// Some of the windows machines in the CI need more time to establish connection -const autoSelectFamilyAttemptTimeout = common.platformTimeout(common.isWindows ? 1500 : 250); +// Some of the machines in the CI need more time to establish connection +const autoSelectFamilyAttemptTimeout = common.defaultAutoSelectFamilyAttemptTimeout; function _lookup(resolver, hostname, options, cb) { resolver.resolve(hostname, 'ANY', (err, replies) => { diff --git a/test/parallel/test-tls-connect-hints-option.js b/test/parallel/test-tls-connect-hints-option.js index 4653e0a1418bbd..6abcf19402efcb 100644 --- a/test/parallel/test-tls-connect-hints-option.js +++ b/test/parallel/test-tls-connect-hints-option.js @@ -25,7 +25,7 @@ tls.connect({ port: 42, lookup: common.mustCall((host, options) => { assert.strictEqual(host, 'localhost'); - assert.deepStrictEqual(options, { family: undefined, hints }); + assert.deepStrictEqual(options, { family: undefined, hints, all: true }); }), hints }); diff --git a/test/sequential/test-http-econnrefused.js b/test/sequential/test-http-econnrefused.js index 7f8c2cee56c4c8..4d0f7a174e0f80 100644 --- a/test/sequential/test-http-econnrefused.js +++ b/test/sequential/test-http-econnrefused.js @@ -32,7 +32,6 @@ const common = require('../common'); const http = require('http'); const assert = require('assert'); - const server = http.createServer(function(req, res) { let body = ''; @@ -135,7 +134,10 @@ function ping() { console.log(`Error making ping req: ${error}`); hadError = true; assert.ok(!gotEnd); - afterPing(error.message); + + // Family autoselection might be skipped if only a single address is returned by DNS. + const actualError = Array.isArray(error.errors) ? error.errors[0] : error; + afterPing(actualError.message); }); } diff --git a/test/sequential/test-net-better-error-messages-port.js b/test/sequential/test-net-better-error-messages-port.js index c21427ee395139..4f09b86d75a8d2 100644 --- a/test/sequential/test-net-better-error-messages-port.js +++ b/test/sequential/test-net-better-error-messages-port.js @@ -7,8 +7,11 @@ const c = net.createConnection(common.PORT); c.on('connect', common.mustNotCall()); -c.on('error', common.mustCall(function(e) { - assert.strictEqual(e.code, 'ECONNREFUSED'); - assert.strictEqual(e.port, common.PORT); - assert.match(e.address, /^(127\.0\.0\.1|::1)$/); +c.on('error', common.mustCall(function(error) { + // Family autoselection might be skipped if only a single address is returned by DNS. + const failedAttempt = Array.isArray(error.errors) ? error.errors[0] : error; + + assert.strictEqual(failedAttempt.code, 'ECONNREFUSED'); + assert.strictEqual(failedAttempt.port, common.PORT); + assert.match(failedAttempt.address, /^(127\.0\.0\.1|::1)$/); })); diff --git a/test/sequential/test-net-connect-econnrefused.js b/test/sequential/test-net-connect-econnrefused.js index 7dec16d2326dd1..c55a9ebe24cc55 100644 --- a/test/sequential/test-net-connect-econnrefused.js +++ b/test/sequential/test-net-connect-econnrefused.js @@ -40,9 +40,12 @@ const server = net.createServer().listen(0, common.mustCall(() => { function pummel() { let pending; for (pending = 0; pending < ATTEMPTS_PER_ROUND; pending++) { - net.createConnection(port).on('error', function(err) { + net.createConnection({ port, autoSelectFamily: false }).on('error', function(error) { + // Family autoselection might be skipped if only a single address is returned by DNS. + const actualError = Array.isArray(error.errors) ? error.errors[0] : error; + console.log('pending', pending, 'rounds', rounds); - assert.strictEqual(err.code, 'ECONNREFUSED'); + assert.strictEqual(actualError.code, 'ECONNREFUSED'); if (--pending > 0) return; if (rounds === ROUNDS) return check(); rounds++; diff --git a/test/sequential/test-net-reconnect-error.js b/test/sequential/test-net-reconnect-error.js index 93d8f3673abda4..1fc26bb1018042 100644 --- a/test/sequential/test-net-reconnect-error.js +++ b/test/sequential/test-net-reconnect-error.js @@ -30,8 +30,11 @@ const c = net.createConnection(common.PORT); c.on('connect', common.mustNotCall('client should not have connected')); -c.on('error', common.mustCall((e) => { - assert.strictEqual(e.code, 'ECONNREFUSED'); +c.on('error', common.mustCall((error) => { + // Family autoselection might be skipped if only a single address is returned by DNS. + const actualError = Array.isArray(error.errors) ? error.errors[0] : error; + + assert.strictEqual(actualError.code, 'ECONNREFUSED'); }, N + 1)); c.on('close', common.mustCall(() => {