net: set ADDRCONFIG DNS hint in connections

cjihrig · jasnell · commit aa395b78fe25 · 2016-04-26T12:16:24.000-07:00
b85a50b removed the implicit setting of DNS hints when creating a connection. This caused some of the pi2 machines to become flaky. This commit restores the implicit dns.ADDRCONFIG hint, but not dns.V4MAPPED. Fixes: #6133 PR-URL: #6281 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Claudio Rodriguez <cjrodr@yahoo.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com> Reviewed-By: Saúl Ibarra Corretgé <saghul@gmail.com>
diff --git a/lib/net.js b/lib/net.js
@@ -949,6 +949,10 @@ function lookupAndConnect(self, options) {
     hints: options.hints || 0
   };
 
+  if (dnsopts.family !== 4 && dnsopts.family !== 6 && dnsopts.hints === 0) {
+    dnsopts.hints = dns.ADDRCONFIG;
+  }
+
   debug('connect: find host ' + host);
   debug('connect: dns options', dnsopts);
   self._host = host;
