dfds / dafda

.NET Kafka client library
7 stars 18 forks source link

Bufgix/producing traceparent propagation #88

Closed toizo closed 7 months ago

toizo commented 7 months ago

The way how Traceparent header was built had an issue such that the started activity/span id was not included. This PR fixed that, among others:

toizo commented 7 months ago

Initially, there was an issue with some serialization tests. This is because some tests use a static invocation of ActivitySource.AddActivityListener(...) that affects whether traceparent and baggage message headers included in the envelope.

I did not find any good way of isolating that static class, so I decided to make the serialization tests part of a same test collection - this prevents them running in parallel.