Retry on 413 (actions#540)

gndelia · Aug 4, 2020 · b2cba16 · b2cba16
1 parent e3c6237
commit b2cba16
Show file tree

Hide file tree

Showing 5 changed files with 9 additions and 3 deletions.
diff --git a/packages/artifact/RELEASES.md b/packages/artifact/RELEASES.md
@@ -33,3 +33,7 @@
 
 - Increase chunk size during upload from 4MB to 8MB
 - Improve user-agent strings during API calls to help internally diagnose issues
+
+### 0.3.4
+
+- Retry in the event of a 413 response
diff --git a/packages/artifact/__tests__/util.test.ts b/packages/artifact/__tests__/util.test.ts
@@ -192,6 +192,7 @@ describe('Utils', () => {
     expect(utils.isRetryableStatusCode(HttpCodes.OK)).toEqual(false)
     expect(utils.isRetryableStatusCode(HttpCodes.NotFound)).toEqual(false)
     expect(utils.isRetryableStatusCode(HttpCodes.Forbidden)).toEqual(false)
+    expect(utils.isRetryableStatusCode(413)).toEqual(true) // Payload Too Large
   })
 
   it('Test Throttled Status Code', () => {

diff --git a/packages/artifact/package-lock.json b/packages/artifact/package-lock.json
diff --git a/packages/artifact/package.json b/packages/artifact/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@actions/artifact",
-  "version": "0.3.3",
+  "version": "0.3.4",
   "preview": true,
   "description": "Actions artifact lib",
   "keywords": [

diff --git a/packages/artifact/src/internal/utils.ts b/packages/artifact/src/internal/utils.ts
@@ -74,7 +74,8 @@ export function isRetryableStatusCode(statusCode?: number): boolean {
     HttpCodes.BadGateway,
     HttpCodes.ServiceUnavailable,
     HttpCodes.GatewayTimeout,
-    HttpCodes.TooManyRequests
+    HttpCodes.TooManyRequests,
+    413 // Payload Too Large
   ]
   return retryableStatusCodes.includes(statusCode)
 }