authn-server OpenTelemetry

Opentracing integration would be great

http://opentracing.io/

Jun 15 '18 03:06 k1ng440

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

Jul 08 '23 16:07 AlexCuse

I agree. OpenTelemetry is the new standard.

Jul 08 '23 16:07 k1ng440

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?

Jul 08 '23 17:07 AlexCuse

Hardest part is that there isn't much context usage in the internal apis currently so span propagation is tough.

I remember trying to implement opentracing back in 2018 but I gave up because of this.

Jul 10 '23 08:07 k1ng440