search

DataDog / opentelemetry-mapping-go

Go modules that implement OpenTelemetry-to-Datadog mapping for all telemetry signals
Apache License 2.0
13 stars 6 forks source link

Fix panics on invalid sized trace & span IDs #340

Closed kevinnoel-be closed 1 month ago

kevinnoel-be commented 1 month ago

Fix for https://github.com/open-telemetry/opentelemetry-collector-contrib/issues/33566

kevinnoel-be commented 1 month ago

Thanks for the PR, it seems like there are some tests failing: https://github.com/DataDog/opentelemetry-mapping-go/actions/runs/9515937620/job/26238071289?pr=340#step:6:62 would you be able to take a look at those?

Oops, sure I'll check that further on my side. Sorry for the noise 😅

kevinnoel-be commented 1 month ago

Should be better now, this is how some upstream process parses it.