-
Notifications
You must be signed in to change notification settings - Fork 110
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
OpenTelemetry #56
Comments
Happy to open a PR for this, assume we should use opentelemetry at this point https://github.com/open-telemetry/opentelemetry-go/tree/main/trace |
I agree. OpenTelemetry is the new standard. |
Hardest part is that there isn't much context usage in the internal apis currently so span propagation is tough. It's easy enough to add but makes for a large change. Guessing db / redis actions and bcrypt operations are probably most interesting to get spans around did you have anything else in mind? |
I remember trying to implement opentracing back in 2018 but I gave up because of this. |
Opentracing integration would be great
http://opentracing.io/
The text was updated successfully, but these errors were encountered: