-
Notifications
You must be signed in to change notification settings - Fork 310
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add an Ox/direct-style-compatible OpenTelemetry tracing backend #2314
Comments
/bounty $750 |
💎 $1,000 bounty • SoftwareMillSteps to solve:
Thank you for contributing to softwaremill/sttp! Add a bounty • Share on socials
|
Once #2292 is done I will jump here to tackle this. |
As we are getting first PRs for the bounties, I've published our "How to prepare a good PR" guide. I should have probably done this right away, sorry! :) |
Hey @adamw is this still available ? |
@stdthoth it's in WIP. Will raise PR soon |
@stdthoth as long as a PR isn't merged it is available - I've setup attempts (with Anyway, the first (good!) PR to get merged wins :) |
I haven't attempted due to algora bot limits the attempts and I haven't done this to make sure not make any messy with algora bot 😀 Anyway /attempt #2314
|
@varshith257: Reminder that in 7 days the bounty will become up for grabs, so please submit a pull request before then 🙏 |
The bounty is up for grabs! Everyone is welcome to |
/attempt #2314
|
@stdthoth: Reminder that in 7 days the bounty will become up for grabs, so please submit a pull request before then 🙏 |
The bounty is up for grabs! Everyone is welcome to |
Currently sttp-client only ships with OT+ZIO tracing integration.
The goal of the issue is to create a new module,
opentelemetry-tracing-sync-backend
, which will provide integration with blocking/direct-style OpenTelemetry tracing, as well as reference documentation showcasing how to use the module.Moreover, a first step is to suggest a way of propagating the context (setting appropriate headers, including identifiers/correlation ids) that is compatible with virtual threads & scoped values, as used by Loom and Ox. Note that the the OT Java libraries often rely on
ThreadLocal
s, which are not Loom-compatible.See also softwaremill/tapir#4103
The text was updated successfully, but these errors were encountered: