avoid unnecessary wrapping

Author: duncanista

bottlecap/src/lifecycle/invocation/processor.rs

@@ -1251,7 +1251,8 @@ impl Processor {
             }
 
             // Extract _dd.p.* propagation tags from x-datadog-tags header
-            tags = extract_propagation_tags(&headers);
+            let carrier_tags = headers.get(DATADOG_TAGS_KEY).map_or("", String::as_str);
+            tags = crate::traces::propagation::extract_propagation_tags(carrier_tags);
         }
 
         // We should always use the generated span id from the tracer
@@ -1359,11 +1360,6 @@ impl Processor {
     }
 }
 
-fn extract_propagation_tags(carrier: &HashMap<String, String>) -> HashMap<String, String> {
-    let carrier_tags = carrier.get(DATADOG_TAGS_KEY).map_or("", String::as_str);
-    crate::traces::propagation::extract_propagation_tags(carrier_tags)
-}
-
 #[cfg(test)]
 #[allow(clippy::unwrap_used)]
 mod tests {

bottlecap/src/lifecycle/invocation/triggers/step_function_event.rs

@@ -151,7 +151,7 @@ impl StepFunctionEvent {
                     .parse()
                     .unwrap_or(Self::generate_trace_id(self.execution.id.clone()).0);
 
-                let tags = extract_propagation_tags(trace_tags);
+                let tags = crate::traces::propagation::extract_propagation_tags(trace_tags);
 
                 (lo_tid, tags)
             } else {
@@ -239,10 +239,6 @@ impl StepFunctionEvent {
     }
 }
 
-fn extract_propagation_tags(tags_str: &str) -> HashMap<String, String> {
-    crate::traces::propagation::extract_propagation_tags(tags_str)
-}
-
 impl ServiceNameResolver for StepFunctionEvent {
     fn get_specific_identifier(&self) -> String {
         String::new()