libdatadog update to d7b2aad3 [no-ci-feedback]

Automated update by CI pipeline https://gitlab.ddbuild.io/DataDog/apm-reliability/dd-trace-php/-/pipelines/122744890

Full CI result: ❌ 176 job(s) failed

Author: github-actions[bot]

components-rs/Cargo.toml

@@ -23,6 +23,7 @@ libdd-data-pipeline = { path = "../libdatadog/libdd-data-pipeline" }
 libdd-tinybytes = { path = "../libdatadog/libdd-tinybytes" }
 libdd-trace-utils = { path = "../libdatadog/libdd-trace-utils" }
 libdd-trace-stats = { path = "../libdatadog/libdd-trace-stats" }
+libdd-trace-protobuf = { path = "../libdatadog/libdd-trace-protobuf" }
 libdd-crashtracker-ffi = { path = "../libdatadog/libdd-crashtracker-ffi", default-features = false, features = ["collector"] }
 libdd-library-config-ffi = { path = "../libdatadog/libdd-library-config-ffi", default-features = false }
 spawn_worker = { path = "../libdatadog/spawn_worker" }

components-rs/remote_config.rs

@@ -92,6 +92,9 @@ pub struct RemoteConfigState {
     manager: RemoteConfigManager,
     live_debugger: LiveDebuggerState,
     dynamic_config: DynamicConfig,
+    /// Service name of the currently tracked `Target`. Tracked here because
+    /// `libdd_remote_config::Target` no longer exposes its fields publicly.
+    service: String,
 }
 
 #[repr(C)]
@@ -223,6 +226,7 @@ pub unsafe extern "C" fn ddog_init_remote_config_state(
             ..Default::default()
         },
         dynamic_config: Default::default(),
+        service: String::new(),
     })
 }
 
@@ -639,13 +643,20 @@ pub extern "C" fn ddog_remote_configs_service_env_change(
     tags: &libdd_common_ffi::Vec<Tag>,
     process_tags: &libdd_common_ffi::Vec<Tag>,
 ) -> bool {
-    let new_target = Target {
-        service: service.to_utf8_lossy().to_string(),
-        env: env.to_utf8_lossy().to_string(),
-        app_version: version.to_utf8_lossy().to_string(),
-        tags: tags.as_slice().to_vec(),
-        process_tags: process_tags.as_slice().to_vec(),
-    };
+    let service = service.to_utf8_lossy().to_string();
+    // `Target`'s fields are private; `Target::new` expects `tags`/`process_tags` as
+    // already-formatted `"key:value"` strings (see libdd-remote-config).
+    let new_target = Target::new(
+        service.clone(),
+        env.to_utf8_lossy().to_string(),
+        version.to_utf8_lossy().to_string(),
+        tags.as_slice().iter().map(|t| t.to_string()).collect(),
+        process_tags.as_slice().iter().map(|t| t.to_string()).collect(),
+    );
+
+    // `Target` no longer exposes its `service` field, so keep track of it ourselves
+    // for consumers such as `ddog_send_debugger_diagnostics`.
+    remote_config.service = service;
 
     if let Some(target) = remote_config.manager.get_target() {
         if **target == new_target {
@@ -777,12 +788,7 @@ pub unsafe extern "C" fn ddog_send_debugger_diagnostics<'a>(
     probe: &'a Probe,
     timestamp: u64,
 ) -> MaybeError {
-    let service = Cow::Borrowed(
-        remote_config_state
-            .manager
-            .get_target()
-            .map_or("", |t| t.service.as_str()),
-    );
+    let service = Cow::Borrowed(remote_config_state.service.as_str());
     let mut payload = ddog_debugger_diagnostics_create_unboxed(
         probe,
         service,

components-rs/stats.rs

@@ -10,6 +10,7 @@
 use crate::trace_filter;
 use datadog_ipc::shm_stats::{OwnedShmSpanInput, ShmSpanConcentrator, ShmSpanInput, MAX_PEER_TAGS};
 use datadog_sidecar::service::blocking::{add_span_to_concentrator, SidecarTransport};
+use libdd_trace_protobuf::pb;
 use libdd_trace_stats::span_concentrator::FixedAggregationKey;
 use libdd_common_ffi::slice::{AsBytes, CharSlice};
 use std::collections::HashMap;
@@ -148,7 +149,11 @@ fn build_fixed_key<'a>(span: &'a PhpSpanStats<'a>) -> FixedAggregationKey<&'a st
         http_endpoint: extract_http_endpoint(span),
         http_status_code: extract_http_status_code(span),
         is_synthetics_request: is_synthetics_request(span),
-        is_trace_root: span.is_trace_root,
+        is_trace_root: if span.is_trace_root {
+            pb::Trilean::True
+        } else {
+            pb::Trilean::False
+        },
         grpc_status_code: extract_grpc_status_code(span),
         service_source: char_slice_str(span.service_source),
     }