@autofleet/network 1.4.8 → 1.4.10

package/index.js

@@ -42,12 +42,27 @@ const defaultSettings = {
 
 const createRequestString = (request) => `[${(request.method || '').toUpperCase()}] ${request.baseURL}${request.url}`;
 
+/*
+  The default free socket keepalive timeout, in milliseconds.
+  Setting to 5000 (5 seconds) as default because out grace period is 10 seconds
+  and we want to make sure we don't have any socket issues.
+  See https://www.npmjs.com/package/agentkeepalive
+
+  There is also autoscaling reason for that, if we dont timeout at some point,
+  new pods that were auto scaled will not be used.
+*/
+const FREE_SOCKET_TIMEOUT = process.env.FREE_SOCKET_TIMEOUT || 5000;
+
 module.exports = class Network {
   constructor(settings = {}) {
     this.settings = merge(defaultSettings, settings);
     if (this.settings.keepAlive) {
-      const keepaliveAgent = new Agent();
-      const keepaliveHttpsAgent = new HttpsAgent();
+      const keepaliveAgent = new Agent({
+        freeSocketTimeout: FREE_SOCKET_TIMEOUT,
+      });
+      const keepaliveHttpsAgent = new HttpsAgent({
+        freeSocketTimeout: FREE_SOCKET_TIMEOUT,
+      });
       this.settings.httpAgent = keepaliveAgent;
       this.settings.httpsAgent = keepaliveHttpsAgent;
       delete this.settings.keepAlive;
@@ -96,11 +111,13 @@ module.exports = class Network {
       return response;
     }, (error) => {
       if(error.request && error.request._currentRequest &&
-          error.request._currentRequest.reusedSocket && error.code === 'ECONNRESET') {
+          error.request._currentRequest.reusedSocket &&
+          ['ECONNRESET', 'EPIPE'].includes(error.code)
+        ) {
         // See https://www.npmjs.com/package/agentkeepalive
         // Support req.reusedSocket
         // https://code-examples.net/en/q/28a8069
-        logger.warn('ECONNRESET issue, will retry', {
+        logger.warn(`${error.code} issue, will retry`, {
           req: error.request._currentRequest._currentUrl,
           // method: error.request._currentRequest._options.method,
         })
@@ -164,7 +181,7 @@ module.exports = class Network {
 
   async getAllPagesFromQueryEndpoint(url, options = {}) {
     let currentResult = [];
-    
+
     let moreResultsToLoad = true;
     let page = 1;
     while(moreResultsToLoad && page < 100) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@autofleet/network",
-  "version": "1.4.8",
+  "version": "1.4.10",
   "description": "",
   "main": "index.js",
   "scripts": {