Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Socket hangups #858

Merged
merged 2 commits into from

2 participants

@temas

We've seen an increase in number of socket hangups that are happening recently. Research suggests two routes, the max sockets increase and the newer knox has changes specifically addressing hangups.

@temas

For reference on why I made these changes take a look at Automattic/knox#116. maxSockets is referenced as well as a displeasure at the agent in general. Reportedly the new knox version helped with the hangups.

@dizzyd

+1

@temas temas merged commit 573b948 into master

1 check passed

Details default The Travis build passed
@temas temas deleted the socket-hangups branch
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
This page is out of date. Refresh to see the latest.
Showing with 3 additions and 3 deletions.
  1. +2 −2 hallwayd.js
  2. +1 −1  package.json
View
4 hallwayd.js
@@ -40,8 +40,8 @@ setInterval(function() {
}, lconfig.dnsReapTime);
// Set our globalAgent sockets higher
-http.globalAgent.maxSockets = 800;
-https.globalAgent.maxSockets = 800;
+http.globalAgent.maxSockets = 2048;
+https.globalAgent.maxSockets = 2048;
function startAPIHost(cbDone) {
logger.vital("Starting an API host");
View
2  package.json
@@ -49,7 +49,7 @@
"generic-pool": "2.0.2",
"hiredis": "0.1.x",
"imap": "git+https://github.com/Singly/node-imap.git",
- "knox": "0.0.9",
+ "knox": "0.6.0",
"lodash": "1.0.1",
"mc": "1.0.x",
"mime": "1.2.x",
Something went wrong with that request. Please try again.