-
-
Notifications
You must be signed in to change notification settings - Fork 935
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Set maxSockets to Infinity by default #30
Conversation
Squashed commits. Ready for review. |
|
||
You can read more about in [why pooling is evil](https://github.com/substack/hyperquest#pooling-is-evil). | ||
|
||
To use default agent just pass `undefined` to this option. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Better to recommend null
rather than undefined
.
Rebased. |
Incapsulated Node logic of creating https agent to infinity-agent module. |
var read = require('read-all-stream'); | ||
var timeout = require('timed-out'); | ||
var urlLib = require('url'); | ||
var zlib = require('zlib'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
formatting yak
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
Set maxSockets to Infinity by default
|
\o/ |
Setting agent for requests to new http(s).Agent({maxSockets: Infinity})
This will get same maxSockets value in 0.10 and upcoming 0.12 node as @julien-f suggested.
Closes #22