Skip to content

Conversation

@shikokuchuo
Copy link
Member

Closes #307.

This PR adds OpenTelemtery support by creating an otel tracing span for top-level functions.

Each existing debug message is then added as an 'event' on the span.

Screenshot 2025-11-05 at 12 20 43

Some questions for @jennybc:

  1. Is it 'safe' to log all of these details given the person collecting these can be someone other than the user (e.g. the hash and user email appear just beyond the screenshot cutoff), or do we need to redact some measures.
  2. I've assumed that this should just be enabled whenever tracing is enabled and no opt-in is required.
  3. token_fetch() is the only instrumented function for now - which other ones should we cover?

Thanks!

To do:

  • Update and complete implementation
  • Add tests

shikokuchuo and others added 2 commits November 5, 2025 12:17
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@shikokuchuo shikokuchuo marked this pull request as draft November 5, 2025 12:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add OpenTelemetry Support

1 participant