Skip to content
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

node.js bug with slow socket release workaround #119

Open
wants to merge 13 commits into
base: master
Choose a base branch
from

Conversation

olegabr
Copy link

@olegabr olegabr commented Nov 10, 2015

see http://habrahabr.ru/post/264851/

The problem is: available socket limit is reached. This workaround solves the problem in some scale.

@see http://habrahabr.ru/post/264851/

The problem is: available socket limit is reached. This workaround solves the problem in some scale.
latest node versions to test
we don't need that shit
…request with a single method supplied.

The actual method name requested is given in a first element of the params array in a callback.
 - It can be used for cudtom headers manipulation, e.g. setting cookies on the response object
 - If promiselib is specified in the options, e.g. `{promiselib: Promise}` and no callback is specified, then methodCall would return a promise.
 - And proper fault response generation is implemented with or without code.
 - Tests and documentation included.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants