http: default Agent.getName to 'localhost'

Refactor out the if/else statement checking for option.host. Add whitespace to make concatenation chunks more readable and consistent with the https version of Agent.getName(). PR-URL: #2825 Reviewed-By: Julian Duque <[email protected]> Reviewed-By: Rich Trott <[email protected]>
nodejs · Sep 16, 2015 · 431bf74 · 431bf74
1 parent 0a5f80a
commit 431bf74
Showing 1 changed file with 3 additions and 6 deletions.
diff --git a/lib/_http_agent.js b/lib/_http_agent.js
@@ -94,19 +94,16 @@ Agent.prototype.createConnection = net.createConnection;
 
 // Get the key for a given set of request options
 Agent.prototype.getName = function(options) {
-  var name = '';
-
-  if (options.host)
-    name += options.host;
-  else
-    name += 'localhost';
+  var name = options.host || 'localhost';
 
   name += ':';
   if (options.port)
     name += options.port;
+
   name += ':';
   if (options.localAddress)
     name += options.localAddress;
+
   return name;
 };