fix: Don't let process continue running if the main app thread panics

blue-build · Aug 20, 2024 · 7418f60 · 7418f60
1 parent 50ba091
commit 7418f60
Showing 1 changed file with 10 additions and 1 deletion.
diff --git a/process/signal_handler.rs b/process/signal_handler.rs
@@ -87,7 +87,16 @@ where
     signals.extend(TERM_SIGNALS);
     let mut signals = SignalsInfo::<WithOrigin>::new(signals).expect("Need signal info");
 
-    thread::spawn(app_exec);
+    thread::spawn(|| {
+        let app = thread::spawn(app_exec);
+
+        if matches!(app.join(), Ok(())) {
+            expect_exit::exit_unwind(0);
+        } else {
+            error!("App thread panic!");
+            expect_exit::exit_unwind(1);
+        }
+    });
 
     let mut has_terminal = true;
     for info in &mut signals {