fix: zipkin trace_id and span_id format in ngx_var (#12403)

Author: Baoyuantop

apisix/plugins/zipkin.lua

@@ -223,8 +223,8 @@ function _M.rewrite(plugin_conf, ctx)
                                              to_hex(span_context.trace_id),
                                              to_hex(span_context.span_id),
                                              span_context:get_baggage_item("x-b3-sampled"))
-        ngx_var.zipkin_trace_id = span_context.trace_id
-        ngx_var.zipkin_span_id = span_context.span_id
+        ngx_var.zipkin_trace_id = to_hex(span_context.trace_id)
+        ngx_var.zipkin_span_id = to_hex(span_context.span_id)
     end
 
     if not ctx.opentracing_sample then

t/plugin/mcp-bridge.t

@@ -91,7 +91,7 @@ passed
 
 
 === TEST 3: test mcp client
---- timeout: 15
+--- timeout: 20
 --- exec
 cd t/plugin/mcp && pnpm test 2>&1
 --- no_error_log

t/plugin/zipkin3.t

@@ -55,6 +55,20 @@ _EOC_
             ngx.log(ngx.ERR,"ngx_var.zipkin_context_traceparent:",ngx.var.zipkin_context_traceparent)
         end
 
+        local trace_id = ngx.var.zipkin_trace_id
+        if trace_id == nil or trace_id == '' then
+           ngx.log(ngx.ERR,"ngx_var.zipkin_trace_id is empty")
+        else
+            ngx.log(ngx.ERR,"ngx_var.zipkin_trace_id:",ngx.var.zipkin_trace_id)
+        end
+
+        local span_id = ngx.var.zipkin_span_id
+        if span_id == nil or span_id == '' then
+           ngx.log(ngx.ERR,"ngx_var.zipkin_span_id is empty")
+        else
+            ngx.log(ngx.ERR,"ngx_var.zipkin_span_id:",ngx.var.zipkin_span_id)
+        end
+
         local orig = orig_func(...)
         return orig
     end
@@ -118,7 +132,23 @@ qr/ngx_var.zipkin_context_traceparent:00-\w{32}-\w{16}-01*/
 
 
 
-=== TEST 3: trigger zipkin with disable set variables
+=== TEST 3: trigger zipkin with open set variables
+--- request
+GET /echo
+--- error_log eval
+qr/ngx_var.zipkin_trace_id:\w{32}/
+
+
+
+=== TEST 4: trigger zipkin with open set variables
+--- request
+GET /echo
+--- error_log eval
+qr/ngx_var.zipkin_span_id:\w{16}/
+
+
+
+=== TEST 5: trigger zipkin with disable set variables
 --- extra_yaml_config
 plugins:
     - zipkin