1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

http: default Agent.getName to 'localhost'

Browse Source 
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: https://github.com/nodejs/node/pull/2825
Reviewed-By: Julian Duque <julianduquej@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
This commit is contained in:
Malcolm Ahoy 2015-09-11 14:18:01 -07:00 committed by Rich Trott
parent e7a3ca3d31
commit 7d7941235a
1 changed files with 3 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
lib/_http_agent.js
View File

@ -94,19 +94,16 @@ Agent.prototype.createConnection = net.createConnection;
// Get the key for a given set of request options // Get the key for a given set of request options
Agent.prototype.getName = function(options) { Agent.prototype.getName = function(options) {
var name = ''; var name = options.host || 'localhost';
if (options.host)
name += options.host;
else
name += 'localhost';
name += ':'; name += ':';
if (options.port) if (options.port)
name += options.port; name += options.port;
name += ':'; name += ':';
if (options.localAddress) if (options.localAddress)
name += options.localAddress; name += options.localAddress;
return name; return name;
}; };