-
Notifications
You must be signed in to change notification settings - Fork 422
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 OpenTelemetry direct-style tracing interceptor #4103
Comments
/bounty $750 |
💎 $1,000 bounty • SoftwareMillSteps to solve:
Thank you for contributing to softwaremill/tapir! Add a bounty • Share on socials
|
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! :) |
/attempt #4103
|
@adamw is this module is still needed.would love to work on this if anyone is not working . |
@kernel-loophole I had progress on it. Just I have open PR of other issue and I am waiting to merge it and get into this completed |
@varshith257 hm which PR you refer to? maybe I've missed something:) |
I just made initial changes long before for this issue which I think we primarily need to work on in sttp(softwaremill/sttp#2314 ) and need to test and evaluate my approach. Just thought to complete them (#4057 and https://github.com/softwaremill/sttp/issues/2292one by one rather than solving this set of issues (refactor of OT and this tracing/opentelemetry-tracing-syn issues simultaneously |
/attempt #4103 Options |
Note The user @varshith257 is already attempting to complete issue #4103 and claim the bounty. We recommend checking in on @varshith257's progress, and potentially collaborating, before starting a new solution. |
@hoklims: Reminder that in 7 days the bounty will become up for grabs, so please submit a pull request before then 🙏 |
/attempt #4103 Options |
Note The user @varshith257 is already attempting to complete issue #4103 and claim the bounty. We recommend checking in on @varshith257's progress, and potentially collaborating, before starting a new solution. |
@promisingcoder: Reminder that in 7 days the bounty will become up for grabs, so please submit a pull request before then 🙏 |
Introduce a new module,
tracing/opentelemetry-tracing-sync
, which should include integration with OT's tracing: reading the context from the request and propagating it downstream.The approach should target "direct-style" (that is, the
Identity
"effect type"), and be compatible with Loom/Ox, that is with virtual threads & scoped values. Note that the the OT Java libraries often rely on ThreadLocals, which are not Loom-compatible.The solution should work in conjunction with softwaremill/sttp#2314
The text was updated successfully, but these errors were encountered: