DOH

DeterminateSystems · Aug 23, 2024 · fe32be8 · fe32be8
1 parent c985ebf
commit fe32be8
Show file tree

Hide file tree

Showing 4 changed files with 36 additions and 32 deletions.
diff --git a/dist/index.d.ts b/dist/index.d.ts
diff --git a/dist/index.js b/dist/index.js
diff --git a/dist/index.js.map b/dist/index.js.map
diff --git a/src/index.ts b/src/index.ts
@@ -398,6 +398,8 @@ export abstract class DetSysAction {
         await this.main();
       } else if (this.isPost) {
         await this.post();
+
+        await this.collectBacktraces();
       }
       this.addFact(FACT_ENDED_WITH_EXCEPTION, false);
     } catch (e: unknown) {
@@ -413,22 +415,6 @@ export abstract class DetSysAction {
         actionsCore.setFailed(reportable);
       }
 
-      if (this.isPost) {
-        try {
-          const backtraces = await collectBacktraces(
-            this.actionOptions.binaryNamePrefixes,
-          );
-          if (backtraces.size > 0) {
-            actionsCore.debug(`backtraces identified: ${backtraces.size}`);
-            this.recordEvent(EVENT_BACKTRACES, Object.fromEntries(backtraces));
-          }
-        } catch (innerError: unknown) {
-          actionsCore.debug(
-            `Error collecting backtraces: ${stringifyError(innerError)}`,
-          );
-        }
-      }
-
       const doGzip = promisify(gzip);
 
       const exceptionContext: Map<string, string> = new Map();
@@ -777,6 +763,22 @@ export abstract class DetSysAction {
     }
   }
 
+  private async collectBacktraces(): Promise<void> {
+    try {
+      const backtraces = await collectBacktraces(
+        this.actionOptions.binaryNamePrefixes,
+      );
+      if (backtraces.size > 0) {
+        actionsCore.debug(`backtraces identified: ${backtraces.size}`);
+        this.recordEvent(EVENT_BACKTRACES, Object.fromEntries(backtraces));
+      }
+    } catch (innerError: unknown) {
+      actionsCore.debug(
+        `Error collecting backtraces: ${stringifyError(innerError)}`,
+      );
+    }
+  }
+
   private async preflightRequireNix(): Promise<boolean> {
     let nixLocation: string | undefined;