From a884e861e98a7f9d05e344e1866d7cb144ce77dd Mon Sep 17 00:00:00 2001 From: Julia Ryan Date: Tue, 5 Aug 2025 20:20:42 -0500 Subject: [PATCH] Tag crash reports with panic message and release (#35692) This _should_ allow sentry to associate related panic events with the same issue, but it doesn't change the issue title. I'm still working on figuring out how to set those fields, but in the meantime this should at least associate zed versions with crashes Release Notes: - N/A --- crates/zed/src/reliability.rs | 15 ++++++++++----- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/crates/zed/src/reliability.rs b/crates/zed/src/reliability.rs index df50ecc743..53539699cc 100644 --- a/crates/zed/src/reliability.rs +++ b/crates/zed/src/reliability.rs @@ -554,6 +554,10 @@ async fn upload_previous_panics( .log_err(); } + if MINIDUMP_ENDPOINT.is_none() { + return Ok(most_recent_panic); + } + // loop back over the directory again to upload any minidumps that are missing panics let mut children = smol::fs::read_dir(paths::logs_dir()).await?; while let Some(child) = children.next().await { @@ -598,11 +602,12 @@ async fn upload_minidump( ) .text("platform", "rust"); if let Some(panic) = panic { - form = form.text( - "release", - format!("{}-{}", panic.release_channel, panic.app_version), - ); - // TODO: tack on more fields + form = form + .text( + "sentry[release]", + format!("{}-{}", panic.release_channel, panic.app_version), + ) + .text("sentry[logentry][formatted]", panic.payload.clone()); } let mut response_text = String::new();