Retry http request on ECONNRESET error (#3739)

aws · May 11, 2021 · 649a91d · 649a91d
1 parent fefebc5
commit 649a91d
Show file tree

Hide file tree

Showing 3 changed files with 50 additions and 2 deletions.
diff --git a/.changes/next-release/bugfix-retry-c648c8ae.json b/.changes/next-release/bugfix-retry-c648c8ae.json
@@ -0,0 +1,5 @@
+{
+  "type": "bugfix",
+  "category": "retry",
+  "description": "Retry http request on ECONNRESET error"
+}
diff --git a/lib/http/node.js b/lib/http/node.js
@@ -87,13 +87,17 @@ AWS.NodeHttpClient = AWS.util.inherit({
       stream.abort();
     });
 
-    stream.on('error', function() {
+    stream.on('error', function(err) {
       if (connectTimeoutId) {
         clearTimeout(connectTimeoutId);
         connectTimeoutId = null;
       }
       if (stream.didCallback) return; stream.didCallback = true;
-      errCallback.apply(stream, arguments);
+      if ('ECONNRESET' === err.code || 'EPIPE' === err.code || 'ETIMEDOUT' === err.code) {
+        errCallback(AWS.util.error(err, {code: 'TimeoutError'}));
+      } else {
+        errCallback(err);
+      }
     });
 
     var expect = httpRequest.headers.Expect || httpRequest.headers.expect;

diff --git a/test/node_http_client.spec.js b/test/node_http_client.spec.js