From 268d7547644ab8a9d0c1163299e59a1f5d93f39b Mon Sep 17 00:00:00 2001 From: Konrad Pabjan Date: Tue, 4 Aug 2020 17:36:48 +0200 Subject: [PATCH] Retry on 413 response codes (#108) --- dist/index.js | 9 +++++---- package-lock.json | 6 +++--- package.json | 2 +- 3 files changed, 9 insertions(+), 8 deletions(-) diff --git a/dist/index.js b/dist/index.js index 264f2de..b1a9863 100644 --- a/dist/index.js +++ b/dist/index.js @@ -4052,7 +4052,7 @@ exports.getUploadFileConcurrency = getUploadFileConcurrency; // When uploading large files that can't be uploaded with a single http call, this controls // the chunk size that is used during upload function getUploadChunkSize() { - return 4 * 1024 * 1024; // 4 MB Chunks + return 8 * 1024 * 1024; // 8 MB Chunks } exports.getUploadChunkSize = getUploadChunkSize; // The maximum number of retries that can be attempted before an upload or download fails @@ -6658,7 +6658,7 @@ const upload_gzip_1 = __webpack_require__(647); const stat = util_1.promisify(fs.stat); class UploadHttpClient { constructor() { - this.uploadHttpManager = new http_manager_1.HttpManager(config_variables_1.getUploadFileConcurrency(), 'actions/upload-artifact'); + this.uploadHttpManager = new http_manager_1.HttpManager(config_variables_1.getUploadFileConcurrency(), '@actions/artifact-upload'); this.statusReporter = new status_reporter_1.StatusReporter(10000); } /** @@ -7412,7 +7412,7 @@ const http_manager_1 = __webpack_require__(452); const config_variables_1 = __webpack_require__(401); class DownloadHttpClient { constructor() { - this.downloadHttpManager = new http_manager_1.HttpManager(config_variables_1.getDownloadFileConcurrency(), 'actions/download-artifact'); + this.downloadHttpManager = new http_manager_1.HttpManager(config_variables_1.getDownloadFileConcurrency(), '@actions/artifact-download'); // downloads are usually significantly faster than uploads so display status information every second this.statusReporter = new status_reporter_1.StatusReporter(1000); } @@ -7942,7 +7942,8 @@ function isRetryableStatusCode(statusCode) { http_client_1.HttpCodes.BadGateway, http_client_1.HttpCodes.ServiceUnavailable, http_client_1.HttpCodes.GatewayTimeout, - http_client_1.HttpCodes.TooManyRequests + http_client_1.HttpCodes.TooManyRequests, + 413 // Payload Too Large ]; return retryableStatusCodes.includes(statusCode); } diff --git a/package-lock.json b/package-lock.json index e789485..bc58505 100644 --- a/package-lock.json +++ b/package-lock.json @@ -5,9 +5,9 @@ "requires": true, "dependencies": { "@actions/artifact": { - "version": "0.3.3", - "resolved": "https://registry.npmjs.org/@actions/artifact/-/artifact-0.3.3.tgz", - "integrity": "sha512-sKC1uA5p6064C6Qypmmt6O8iKlpDyMTfqqDlS4/zfJX1Hs8NbbzPLLN81RpewuJPWQNnroeF52w4VCWypbSNaA==", + "version": "0.3.5", + "resolved": "https://registry.npmjs.org/@actions/artifact/-/artifact-0.3.5.tgz", + "integrity": "sha512-y27pBEnUjOqCP2zUf86YkiqGOp1r0C9zUOmGmcxizsHMls0wvk+FJwd+l8JIoukvj1BeBHYP+c+9AEqOt5AqMA==", "dev": true, "requires": { "@actions/core": "^1.2.1", diff --git a/package.json b/package.json index 12511ee..7867d1f 100644 --- a/package.json +++ b/package.json @@ -29,7 +29,7 @@ }, "homepage": "https://github.com/actions/upload-artifact#readme", "devDependencies": { - "@actions/artifact": "^0.3.3", + "@actions/artifact": "^0.3.5", "@actions/core": "^1.2.3", "@actions/glob": "^0.1.0", "@actions/io": "^1.0.2",