open-telemetry / opentelemetry-java

OpenTelemetry Java SDK
https://opentelemetry.io
Apache License 2.0
2.01k stars 839 forks source link

Make StrictContextStorage public #6716

Closed johnpoth closed 1 month ago

johnpoth commented 2 months ago

Is your feature request related to a problem? Please describe. It would be cool to make the StrictContextStorage class public so that other projects can use it in their own tests. Thanks !

Describe the solution you'd like make the StrictContextStorage class public

Describe alternatives you've considered Copy paste

Thanks !

jkwatson commented 2 months ago

I'm not opposed to this idea, but can you not leverage -Dio.opentelemetry.context.enableStrictContext=true to turn it on in tests?

johnpoth commented 2 months ago

@jkwatson thanks for your response! Yes that is great thank you !

trask commented 2 months ago

hi @johnpoth, did you still want to pursue making StringContextStorage public, or does this address your needs and we can close the issue? thanks!

johnpoth commented 1 month ago

@trask I think we can close the issue thank you so much!