rebus-org / Rebus.OpenTelemetry

Other
12 stars 4 forks source link

Why not use original rbs2-corr-id for trace-state/parent-id #15

Open riezebosch opened 1 year ago

riezebosch commented 1 year ago

https://github.com/rebus-org/Rebus/blob/master/Rebus/Messages/Headers.cs#L26

Instead of using another custom header.

zlepper commented 2 months ago

Mainly backwardscompatibility. You never know what might depend on something, so in case messages are being based to/from services that doesn't have open telemetry enabled, we are not going to conflict with any custom logic. Though the question is still valid.

@mookid8000 Would there be anything wrong with co-opting the mentioned header for this?

mookid8000 commented 2 months ago

@zlepper I have no idea, actually. 🙂