search

instana / instana-java-opentracing

Instana Java OpenTracing
MIT License
20 stars 6 forks source link

Limit Header extraction to Instana-relevant header fields #10

Closed jfk-sixt closed 1 year ago

jfk-sixt commented 2 years ago

fixes #3

jfk-sixt commented 2 years ago

One risk is worth mentioning: The fix might break some implicit expectations. When some legacy code uses the Tracer and erroneously relies on the previous accidental header passing (instead of passing headers explicitly, where desired), it might break after the fix.

I think this should be addressed in the release notes.

mrajaian commented 1 year ago

Moved to a new PR as we had to support ByteBufferContext as well, https://github.com/instana/instana-java-opentracing/pull/11

Thank you @jfk-sixt for your contribution