opentelemetry-php icon indicating copy to clipboard operation
opentelemetry-php copied to clipboard

Align trace ID and span ID naming with other SDKs

Open piotrooo opened this issue 1 month ago • 3 comments

What is the expected behavior? The opentelemetry-php/contrib-auto-psr3 project sets traceId and spanId in camelCase:

https://github.com/opentelemetry-php/contrib-auto-psr3/blob/eebb2559a84e9fd653cbf736c2209f44772c8cc2/src/Psr3Instrumentation.php#L57-L58

What is the actual behavior? All other SDKs use trace_id and span_id written in snake_case. Is this intentional, or should it be changed to comply with other SDKs?

Tip: React with 👍 to help prioritize this issue. Please use comments to provide useful context, avoiding +1 or me too, to help us triage it. Learn more here.

piotrooo avatar Oct 27 '25 11:10 piotrooo

I think it's ok to change to snake case, if you want to create a PR for it

brettmc avatar Oct 29 '25 09:10 brettmc

@piotrooo are you planning on doing this work, or should we label it help wanted?

bobstrecansky avatar Dec 03 '25 13:12 bobstrecansky

@piotrooo are you planning on doing this work, or should we label it help wanted?

@bobstrecansky yes, you should label it as help wanted.

piotrooo avatar Dec 03 '25 13:12 piotrooo