Try something: do not call the previous hook

Author: gleocadie

libdd-crashtracker/src/collector/crash_handler.rs

@@ -107,7 +107,7 @@ pub fn register_panic_hook() -> anyhow::Result<()> {
                 }
             }
         }
-        call_previous_panic_hook(panic_info);
+        //call_previous_panic_hook(panic_info);
     }));
     Ok(())
 }
@@ -120,6 +120,7 @@ pub fn register_panic_hook() -> anyhow::Result<()> {
 /// SAFETY:
 ///     Crash-tracking functions are not guaranteed to be reentrant.
 ///     No other crash-handler functions should be called concurrently.
+#[allow(dead_code)]
 fn call_previous_panic_hook(panic_info: &PanicHookInfo<'_>) {
     let old_hook_ptr = PREVIOUS_PANIC_HOOK.load(SeqCst);
     if !old_hook_ptr.is_null() {

libdd-crashtracker/src/crash_info/builder.rs

@@ -453,4 +453,25 @@ mod tests {
         assert_eq!(crash_ping.metadata(), &metadata);
         assert!(crash_ping.message().contains("crash processing started"));
     }
+
+    #[test]
+    fn test_crash_info_builder_to_crash_ping_with_message() {
+        let sig_info = SigInfo::test_instance(42);
+        let metadata = Metadata::test_instance(1);
+        let message = "test message";
+
+        let mut crash_info_builder = CrashInfoBuilder::new();
+        crash_info_builder.with_sig_info(sig_info.clone()).unwrap();
+        crash_info_builder.with_metadata(metadata.clone()).unwrap();
+        crash_info_builder.with_kind(ErrorKind::Panic).unwrap();
+        crash_info_builder.with_message(message.to_string()).unwrap();
+
+        let crash_ping = crash_info_builder.build_crash_ping().unwrap();
+
+        assert!(!crash_ping.crash_uuid().is_empty());
+        assert!(Uuid::parse_str(crash_ping.crash_uuid()).is_ok());
+        assert_eq!(crash_ping.siginfo(), Some(&sig_info));
+        assert_eq!(crash_ping.metadata(), &metadata);
+        assert!(crash_ping.message().contains("crash processing started - test message"));
+    }
 }