From 95703686db88b9693067627763fd7b5054e9998e Mon Sep 17 00:00:00 2001 From: Louis Tricot Date: Tue, 17 Feb 2026 10:06:06 +0100 Subject: [PATCH 1/2] fix(crashtracking): add process_tags to application field --- .../src/crash_info/telemetry.rs | 22 ++++++++++++++++++- 1 file changed, 21 insertions(+), 1 deletion(-) diff --git a/libdd-crashtracker/src/crash_info/telemetry.rs b/libdd-crashtracker/src/crash_info/telemetry.rs index 71a1fd3d0f..764fc4176d 100644 --- a/libdd-crashtracker/src/crash_info/telemetry.rs +++ b/libdd-crashtracker/src/crash_info/telemetry.rs @@ -208,6 +208,7 @@ impl TelemetryCrashUploader { "runtime-id" => runtime_id, "service_version" => service_version, "service" => service_name, + "process_tags" => process_tags, ); let application = Application { @@ -220,6 +221,7 @@ impl TelemetryCrashUploader { .to_owned(), env: env.map(ToOwned::to_owned), service_version: service_version.map(ToOwned::to_owned), + process_tags: process_tags.map(ToOwned::to_owned), ..Default::default() }; @@ -458,6 +460,16 @@ mod tests { .unwrap() } + fn new_test_uploader_with_process_tags(seed: u64, process_tags: &str) -> TelemetryCrashUploader { + let mut metadata = Metadata::test_instance(seed); + metadata.tags.push(format!("process_tags:{process_tags}")); + TelemetryCrashUploader::new( + &metadata, + &Some(Endpoint::from_slice("http://localhost:8126")), + ) + .unwrap() + } + #[test] #[cfg_attr(miri, ignore)] fn test_profiler_config_extraction() { @@ -467,6 +479,7 @@ mod tests { assert_eq!(metadata.application.service_name, "foo"); assert_eq!(metadata.application.service_version.as_deref(), Some("bar")); assert_eq!(metadata.application.language_name, "native"); + assert_eq!(metadata.application.process_tags, None); assert_eq!(metadata.runtime_id, "xyz"); let cfg = t.cfg; assert_eq!( @@ -486,7 +499,10 @@ mod tests { p }; let seed = 1; - let mut t = new_test_uploader(seed); + let mut t = new_test_uploader_with_process_tags( + seed, + "entrypoint.name:cli,entrypoint.type:script", + ); t.cfg .set_host_from_url(&format!("file://{}", output_filename.to_str().unwrap())) @@ -501,6 +517,10 @@ mod tests { assert_eq!(payload["application"]["language_name"], "native"); assert_eq!(payload["application"]["service_name"], "foo"); assert_eq!(payload["application"]["service_version"], "bar"); + assert_eq!( + payload["application"]["process_tags"], + "entrypoint.name:cli,entrypoint.type:script" + ); assert_eq!(payload["request_type"], "logs"); assert_eq!(payload["tracer_time"], 1568898000); assert_eq!(payload["origin"], "Crashtracker"); From a5406d3a5366c991a4336338d28900d46d2ec5e0 Mon Sep 17 00:00:00 2001 From: Louis Tricot Date: Tue, 17 Feb 2026 10:12:03 +0100 Subject: [PATCH 2/2] fix lint --- libdd-crashtracker/src/crash_info/telemetry.rs | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/libdd-crashtracker/src/crash_info/telemetry.rs b/libdd-crashtracker/src/crash_info/telemetry.rs index 764fc4176d..1b94a448af 100644 --- a/libdd-crashtracker/src/crash_info/telemetry.rs +++ b/libdd-crashtracker/src/crash_info/telemetry.rs @@ -460,7 +460,10 @@ mod tests { .unwrap() } - fn new_test_uploader_with_process_tags(seed: u64, process_tags: &str) -> TelemetryCrashUploader { + fn new_test_uploader_with_process_tags( + seed: u64, + process_tags: &str, + ) -> TelemetryCrashUploader { let mut metadata = Metadata::test_instance(seed); metadata.tags.push(format!("process_tags:{process_tags}")); TelemetryCrashUploader::new( @@ -499,10 +502,8 @@ mod tests { p }; let seed = 1; - let mut t = new_test_uploader_with_process_tags( - seed, - "entrypoint.name:cli,entrypoint.type:script", - ); + let mut t = + new_test_uploader_with_process_tags(seed, "entrypoint.name:cli,entrypoint.type:script"); t.cfg .set_host_from_url(&format!("file://{}", output_filename.to_str().unwrap()))