fix: ensure status is Accepted

electron · May 14, 2024 · 7b49573 · 7b49573
1 parent fe55db4
commit 7b49573
Showing 1 changed file with 8 additions and 7 deletions.
diff --git a/src/notarytool.ts b/src/notarytool.ts
@@ -83,21 +83,22 @@ export async function notarizeAndWaitForNotaryTool(opts: NotaryToolStartOptions)
     ];
 
     const result = await spawn('xcrun', notarizeArgs);
-
-    if (result.code === 0) {
-      d('notarization success');
-      return;
-    }
+    const rawOut = result.output.trim();
 
     let parsed: any;
     try {
-      parsed = JSON.parse(result.output.trim());
+      parsed = JSON.parse(rawOut);
     } catch (err) {
       throw new Error(
-        `Failed to notarize via notarytool.  Failed with unexpected result: \n\n${result.output.trim()}`,
+        `Failed to notarize via notarytool.  Failed with unexpected result: \n\n${rawOut}`,
       );
     }
 
+    if (result.code === 0 && parsed.status === 'Accepted') {
+      d('notarization success');
+      return;
+    }
+
     let logOutput: undefined | string;
     if (parsed.id) {
       try {