Skip to content

feat: distributed tracing with OpenTelemetry (issue #22)#200

Closed
luhrhenz wants to merge 1 commit intoSynapse-bridgez:mainfrom
luhrhenz:feature/issue-22-opentelemetry
Closed

feat: distributed tracing with OpenTelemetry (issue #22)#200
luhrhenz wants to merge 1 commit intoSynapse-bridgez:mainfrom
luhrhenz:feature/issue-22-opentelemetry

Conversation

@luhrhenz
Copy link
Copy Markdown
Contributor

closes #22

  • Add opentelemetry, opentelemetry_sdk, opentelemetry-otlp, tracing-opentelemetry deps
  • Add src/telemetry.rs: init_tracer() sets up OTLP batch exporter (or no-op if OTLP_ENDPOINT unset), installs W3C TraceContext propagator
  • Wire OTel layer into tracing-subscriber registry in main()
  • Add OTLP_ENDPOINT to Config (optional env var)
  • Instrument HorizonClient.get_account with #[instrument] + W3C traceparent header injection
  • Instrument webhook handlers: callback, handle_webhook, get_transaction, transaction_callback
  • Instrument TransactionProcessor: process_transaction, requeue_dlq
  • Flush tracer on clean server shutdown

- Add opentelemetry, opentelemetry_sdk, opentelemetry-otlp, tracing-opentelemetry deps
- Add src/telemetry.rs: init_tracer() sets up OTLP batch exporter (or no-op if OTLP_ENDPOINT unset), installs W3C TraceContext propagator
- Wire OTel layer into tracing-subscriber registry in main()
- Add OTLP_ENDPOINT to Config (optional env var)
- Instrument HorizonClient.get_account with #[instrument] + W3C traceparent header injection
- Instrument webhook handlers: callback, handle_webhook, get_transaction, transaction_callback
- Instrument TransactionProcessor: process_transaction, requeue_dlq
- Flush tracer on clean server shutdown
@drips-wave
Copy link
Copy Markdown

drips-wave bot commented Mar 25, 2026

@luhrhenz Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@luhrhenz luhrhenz closed this Mar 25, 2026
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.

Distributed Tracing with OpenTelemetry (Observability)

1 participant