Merge pull request #21 from merc1031/128bit-trace-id

feat: Change trace_id to u128 to follow spec https://www.w3.org/TR/trace-context/#trace-id
krojew · May 20, 2024 · bd521fb · bd521fb
2 parents eac570a + 2b50f99
commit bd521fb
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/src/layer.rs b/src/layer.rs
@@ -18,7 +18,7 @@ use crate::{
 
 #[derive(Copy, Clone)]
 struct TraceContext {
-    pub trace_id: u64,
+    pub trace_id: u128,
 }
 
 struct SpanContext {

diff --git a/src/visitor.rs b/src/visitor.rs
@@ -45,12 +45,14 @@ impl ApmVisitor {
 
 #[derive(Default)]
 #[repr(transparent)]
-pub(crate) struct TraceIdVisitor(pub(crate) Option<u64>);
+pub(crate) struct TraceIdVisitor(pub(crate) Option<u128>);
 
 impl Visit for TraceIdVisitor {
     fn record_i64(&mut self, _field: &Field, _value: i64) {}
 
-    fn record_u64(&mut self, field: &Field, value: u64) {
+    fn record_u64(&mut self, _field: &Field, _value: u64) {}
+
+    fn record_u128(&mut self, field: &Field, value: u128) {
         if field.name() == TRACE_ID_FIELD_NAME {
             self.0 = Some(value);
         }