Skip to content
This repository has been archived by the owner on Apr 22, 2023. It is now read-only.

Commit

Permalink
http: agent takes options instead of host, port pair
Browse files Browse the repository at this point in the history
  • Loading branch information
mikeal authored and ry committed Feb 5, 2011
1 parent 0da96cc commit 2b03ba5
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 9 deletions.
12 changes: 6 additions & 6 deletions lib/http.js
Expand Up @@ -959,10 +959,10 @@ function connectionListener(socket) {
exports._connectionListener = connectionListener;



function Agent(host, port) {
this.host = host;
this.port = port;
function Agent(options) {
this.options = options;
this.host = options.host;
this.port = options.port;

this.queue = [];
this.sockets = [];
Expand Down Expand Up @@ -1228,7 +1228,7 @@ function getAgent(host, port) {
var agent = agents[id];

if (!agent) {
agent = agents[id] = new Agent(host, port);
agent = agents[id] = new Agent({ host: host, port: port });
}

return agent;
Expand All @@ -1248,7 +1248,7 @@ exports.request = function(options, cb) {
if (options.agent === undefined) {
options.agent = getAgent(options.host, options.port);
} else if (options.agent === false) {
options.agent = new Agent(options.host, options.port);
options.agent = new Agent(options);
}
return exports._requestFromAgent(options, cb);
};
Expand Down
4 changes: 1 addition & 3 deletions lib/https.js
Expand Up @@ -26,9 +26,7 @@ exports.createServer = function(opts, requestListener) {
var agents = {};

function Agent(options) {
http.Agent.call(this, options.host, options.port);

this.options = options;
http.Agent.call(this, options);
}
inherits(Agent, http.Agent);

Expand Down

0 comments on commit 2b03ba5

Please sign in to comment.