Fix additional inconsistencies (hostname vs host) in DNS. nodejs#20892

shakeelmohamed · Oct 12, 2018 · 8c1e1d0 · 8c1e1d0
1 parent 0bd9cf4
commit 8c1e1d0
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 10 deletions.
diff --git a/lib/internal/dns/promises.js b/lib/internal/dns/promises.js
@@ -127,34 +127,34 @@ function onlookupservice(err, hostname, service) {
   this.resolve({ hostname, service });
 }
 
-function createLookupServicePromise(host, port) {
+function createLookupServicePromise(hostname, port) {
   return new Promise((resolve, reject) => {
     const req = new GetNameInfoReqWrap();
 
-    req.host = host;
+    req.hostname = hostname;
     req.port = port;
     req.oncomplete = onlookupservice;
     req.resolve = resolve;
     req.reject = reject;
 
-    const err = getnameinfo(req, host, port);
+    const err = getnameinfo(req, hostname, port);
 
     if (err)
-      reject(dnsException(err, 'getnameinfo', host));
+      reject(dnsException(err, 'getnameinfo', hostname));
   });
 }
 
-function lookupService(host, port) {
+function lookupService(hostname, port) {
   if (arguments.length !== 2)
-    throw new ERR_MISSING_ARGS('host', 'port');
+    throw new ERR_MISSING_ARGS('hostname', 'port');
 
-  if (isIP(host) === 0)
-    throw new ERR_INVALID_OPT_VALUE('host', host);
+  if (isIP(hostname) === 0)
+    throw new ERR_INVALID_OPT_VALUE('hostname', hostname);
 
   if (!isLegalPort(port))
     throw new ERR_SOCKET_BAD_PORT(port);
 
-  return createLookupServicePromise(host, +port);
+  return createLookupServicePromise(hostname, +port);
 }
 
 

diff --git a/test/parallel/test-dns.js b/test/parallel/test-dns.js
@@ -258,7 +258,7 @@ dns.lookup('', {
   const err = {
     code: 'ERR_INVALID_OPT_VALUE',
     type: TypeError,
-    message: `The value "${invalidHost}" is invalid for option "host"`
+    message: `The value "${invalidHost}" is invalid for option "hostname"`
   };
 
   common.expectsError(() => {