feat(dashboard-api): HTTP tracing and metrics#2320
Conversation
Align dashboard-api request observability with the main API service so request logs carry trace context and route-level HTTP latency metrics can power Grafana panels. Made-with: Cursor
PR SummaryMedium Risk Overview Reviewed by Cursor Bugbot for commit 6ac100f. Bugbot is set up for automated code reviews on this repo. Configure here. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: a1bf03fa5c
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
packages/dashboard-api/internal/middleware/otel/metrics/middleware.go
Outdated
Show resolved
Hide resolved
packages/dashboard-api/internal/middleware/otel/metrics/config.go
Outdated
Show resolved
Hide resolved
packages/dashboard-api/internal/middleware/otel/metrics/middleware.go
Outdated
Show resolved
Hide resolved
packages/dashboard-api/internal/middleware/otel/metrics/middleware.go
Outdated
Show resolved
Hide resolved
dobrac
left a comment
There was a problem hiding this comment.
If this is the same implementation as in the API, can we please first move it to shared and reuse it here?
packages/dashboard-api/internal/middleware/otel/metrics/middleware.go
Outdated
Show resolved
Hide resolved
…/e2b-dev/infra into dashboard-api-otel-observability
…/e2b-dev/infra into dashboard-api-otel-observability
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit eea4941. Configure here.
matthewlouisbrockman
left a comment
There was a problem hiding this comment.
lotta version changes, seems to still work
|
resolving all bugbot comments since i don't want to change the middleware directly in this pr. it just got shared |

Summary
dashboard-apiso request logs inherittrace_idandspan_iddashboard-api, excluding/healthMade with Cursor