Skip to content

Conversation

@mmatczuk
Copy link
Contributor

@mmatczuk mmatczuk commented Jan 21, 2026

This fixes the invalid parent child relationship in spans. With this patch the spans are rendered correctly
ss-yciRsb15

@mmatczuk mmatczuk requested a review from Jeffail January 21, 2026 18:51
Do not discard instrumented messages returned by tracing.WithChildSpan(...) and restore original context when done.
Based on https://opentelemetry.io/docs/specs/semconv/http/http-spans/

- `http.request.method` - HTTP method (GET, POST, PUT, etc.)
- `url.full` - Complete request URL
- `server.address` - Hostname from URL
- `server.port` - Port number (only if explicitly specified in URL)
- `network.protocol.version` - HTTP version (HTTP/1.1, etc.)
- `url.scheme` - URL scheme (http/https)
- `user_agent.original` - User-Agent header value (if present)

- `http.response.status_code` - HTTP status code (200, 404, etc.)
- `error.type` - Error classification (status code, "timeout", "canceled", "request_error")
- `http.request.resend_count` - Number of retry attempts
@mmatczuk mmatczuk requested a review from weeco January 22, 2026 15:18
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.

4 participants