Add artifact upload error handling

ncipollo · Jan 3, 2020 · f367946 · f367946
1 parent b0c22f0
commit f367946
Show file tree

Hide file tree

Showing 2 changed files with 26 additions and 6 deletions.
diff --git a/lib/ArtifactUploader.js b/lib/ArtifactUploader.js
@@ -8,15 +8,29 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
         step((generator = generator.apply(thisArg, _arguments || [])).next());
     });
 };
+var __importStar = (this && this.__importStar) || function (mod) {
+    if (mod && mod.__esModule) return mod;
+    var result = {};
+    if (mod != null) for (var k in mod) if (Object.hasOwnProperty.call(mod, k)) result[k] = mod[k];
+    result["default"] = mod;
+    return result;
+};
 Object.defineProperty(exports, "__esModule", { value: true });
+const core = __importStar(require("@actions/core"));
 class GithubArtifactUploader {
     constructor(releases) {
         this.releases = releases;
     }
     uploadArtifacts(artifacts, uploadUrl) {
         return __awaiter(this, void 0, void 0, function* () {
             artifacts.forEach((artifact) => __awaiter(this, void 0, void 0, function* () {
-                yield this.releases.uploadArtifact(uploadUrl, artifact.contentLength, artifact.contentType, artifact.readFile(), artifact.name);
+                try {
+                    yield this.releases.uploadArtifact(uploadUrl, artifact.contentLength, artifact.contentType, artifact.readFile(), artifact.name);
+                }
+                catch (error) {
+                    const message = `Failed to upload artifact ${artifact.name}. Does it already exist?`;
+                    core.warning(message);
+                }
             }));
         });
     }

diff --git a/src/ArtifactUploader.ts b/src/ArtifactUploader.ts
@@ -1,3 +1,4 @@
+import * as core from '@actions/core';
 import { Artifact } from "./Artifact";
 import { Releases } from "./Releases";
 
@@ -14,11 +15,16 @@ export class GithubArtifactUploader implements ArtifactUploader {
 
     async uploadArtifacts(artifacts: Artifact[], uploadUrl: string) {
         artifacts.forEach(async artifact => {
-            await this.releases.uploadArtifact(uploadUrl,
-                artifact.contentLength,
-                artifact.contentType,
-                artifact.readFile(),
-                artifact.name)
+            try {
+                await this.releases.uploadArtifact(uploadUrl,
+                    artifact.contentLength,
+                    artifact.contentType,
+                    artifact.readFile(),
+                    artifact.name)
+            } catch(error) {
+                const message = `Failed to upload artifact ${artifact.name}. Does it already exist?`
+                core.warning(message)
+            }
         });
     }
 }