rm-hull / dot-block / 27073968791

61%

push

github

feat: add OpenTelemetry tracing to DNS requests (#140)

* feat: add OpenTelemetry tracing to DNS requests

- Integrated `go.opentelemetry.io` for distributed tracing.
- Added a `TracingHandler` to inject `trace_id` and `span_id` into JSON
logs.
- Instrumented `DNSDispatcher` methods (`HandleDNSRequest`,
`processQuestion`, `resolveUpstream`, `forwardQuery`) to capture request
 lifecycles and latency.
- Configured OTLP exporter for trace collection.

* feat: add distributed tracing support via OpenTelemetry

- Integrated OpenTelemetry (OTel) for end-to-end DNS request tracing.
- Added support for OTLP gRPC exports.
- Introduced `OTEL_SAMPLING_RATIO` and `OTEL_EXPORTER_OTLP_ENDPOINT`
environment variables.
- Configured parent-based sampling to improve observability while
managing data volume.

* style: fix indentation in ConnPool struct

* Update internal/telemetry/telemetry.go

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>

* Update internal/forwarder/dispatcher.go

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>

* refactor: add timeout to tracer shutdown and set default OTLP

- Wrap tracer shutdown in a 5-second context timeout to prevent
  hanging during application exit.
- Define a default OTLP endpoint ("localhost:4317") when the
  `OTEL_EXPORTER_OTLP_ENDPOINT` environment variable is unset.

* feat: configure OTLP exporter with insecure connection

Explicitly set the OTLP endpoint and enable insecure mode for the GRPC
exporter to ensure connectivity to local collectors without TLS
handshake overhead.

* fix: handle nil DNS requests and OTLP exporter errors

- Add safety check for nil `req` in `DNSDispatcher` to prevent panics.
- Return error if OTLP exporter creation fails in
`telemetry.InitTracer`.

* fix: set span status on error in dispatcher

Ensure OpenTelemetry spans are correctly marked as failed when an
upstream
DNS query results in ... (continued)

62 of 147 new or added lines in 6 files covered. (42.18%)

841 of 1809 relevant lines covered (46.49%)

965.34 hits per line