Optimize extract_sentrytrace_data

The optimization adds length checks before expensive string formatting operations. Specifically:

**Key Changes:**
- Added `len(trace_id) != 32` check before `"{:032x}".format(int(trace_id, 16))`
- Added `len(parent_span_id) != 16` check before `"{:016x}".format(int(parent_span_id, 16))`

**Why It's Faster:**
The original code always performed string-to-int conversion and formatting, even when the trace_id/span_id were already properly formatted. The optimization skips these expensive operations when the strings are already the correct length (32 hex chars for trace_id, 16 for span_id).

The `int(trace_id, 16)` and `"{:032x}".format()` operations are computationally expensive, involving:
- Hexadecimal string parsing
- Integer conversion 
- String formatting with zero-padding

**Performance Impact:**
Test results show the optimization is most effective when trace IDs and span IDs are already properly formatted (which is common in production). Cases like `test_valid_full_header` show 51.6% speedup, and `test_missing_trace_id` shows 65.9% speedup. The optimization has minimal overhead for cases where formatting is still needed, with only small gains (1-7%) for malformed inputs.

This is particularly valuable for high-throughput tracing scenarios where most headers contain well-formatted trace data.

Author: codeflash-ai[bot]

sentry_sdk/tracing_utils.py

@@ -327,9 +327,9 @@ def extract_sentrytrace_data(header):
     trace_id, parent_span_id, sampled_str = match.groups()
     parent_sampled = None
 
-    if trace_id:
+    if trace_id and len(trace_id) != 32:
         trace_id = "{:032x}".format(int(trace_id, 16))
-    if parent_span_id:
+    if parent_span_id and len(parent_span_id) != 16:
         parent_span_id = "{:016x}".format(int(parent_span_id, 16))
     if sampled_str:
         parent_sampled = sampled_str != "0"
@@ -527,7 +527,9 @@ def _fill_sample_rand(self):
             )
             return
 
-        self.dynamic_sampling_context["sample_rand"] = f"{sample_rand:.6f}"  # noqa: E231
+        self.dynamic_sampling_context["sample_rand"] = (
+            f"{sample_rand:.6f}"  # noqa: E231
+        )
 
     def _sample_rand(self):
         # type: () -> Optional[str]