remove change to behaviour on 500 errors

github · May 1, 2020 · e52e34b · e52e34b
1 parent cffc0f7
commit e52e34b
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git a/lib/upload-lib.js b/lib/upload-lib.js
diff --git a/src/upload-lib.ts b/src/upload-lib.ts
@@ -83,6 +83,12 @@ async function uploadPayload(payload) {
             // Sleep for the backoff period
             await new Promise(r => setTimeout(r, backoffPeriods[attempt] * 1000));
 
+        } else if (res.message.statusCode === 500) {
+            // If the upload fails with 500 then we assume it is a temporary problem
+            // with turbo-scan and not an error that the user has caused or can fix.
+            // We avoid marking the job as failed to avoid breaking CI workflows.
+            core.error('Upload failed (' + requestID + '): ' + await res.readBody());
+
         } else {
             core.setFailed('Upload failed (' + requestID + '): ' + await res.readBody());
         }