Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HTTP metrics: reporting additional error reason when response was not received #204

Closed
lmolkova opened this issue Jul 24, 2023 · 0 comments · Fixed by #205
Closed

HTTP metrics: reporting additional error reason when response was not received #204

lmolkova opened this issue Jul 24, 2023 · 0 comments · Fixed by #205
Assignees

Comments

@lmolkova
Copy link
Contributor

Original discussion: open-telemetry/opentelemetry-specification#3243 (comment)

Problem:

  • HTTP metrics have http.response.status_code attribute to measure error rate/count:

    • When an error happens before the response is received (SSL, DNS, timeout, connection refused, etc), no HTTP status code is reported.
    • Essentially, the lack of http.response.status_code means an error
  • It's helpful to know more error details (timeout, DNS, SSL, etc) when no response is received. We can record it on spans in description, and in logs as an exceptions, but there is no way to record it on metrics.

  • Adding a new attribute like this to metrics would be a breaking change as we'll split 'error' bucket into multiple new time series.

Proposal:

  • introduce HTTP-specific extendable status code to represent an error reason.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging a pull request may close this issue.

2 participants