Bug: No telemetry recorded for failed requests
Maintainer thường phản hồi trong vòng 1 ngày
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức phù hợp với người mới
- 48/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- java
- Lĩnh vực
- observability
Hướng nghiên cứu
Bắt đầu trong src/main/java/dev/openfga/sdk/api/client/HttpRequestAttempt.java, tập trung vào việc ghi nhận metric trên đường dẫn thành công quanh các dòng 211-221 và các đường dẫn lỗi kết thúc đối với các lần thử lại đã hết và các lỗi không thể thử lại. Đảm bảo requestDuration và queryDuration được ghi nhận cho cả request thất bại lẫn request thành công, đồng thời giữ nguyên sự phân biệt hiện có theo mã trạng thái.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Summary
In src/main/java/dev/openfga/sdk/api/client/HttpRequestAttempt.java, when a request fails permanently (max retries exceeded, non-retryable error), requestDuration and queryDuration metrics are never recorded. Only the success path (~line 211-221) records metrics.
Impact
- Error rates are invisible via telemetry
- Request duration for failed calls is not tracked
- Users cannot build alerting/dashboards on error patterns
- Failed requests are a silent gap in observability
Proposed Fix
Record requestDuration on all terminal paths (success and failure), not just success. The http.response.status_code attribute already distinguishes success from failure, so existing dashboards filtering by status code will continue to work correctly.
This could be done with a try/finally pattern or by recording metrics just before throwing in the error paths.
Related
- PR #290 (telemetry instance sharing fix)
- Ngôn ngữ chính
- Java
- Star
- 55
- Fork
- 26
- Merge trung bình
- 1 ngày 22 giờ
- Pull request đã merge (30 ngày)
- 5
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Không có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của openfga/java-sdk
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Make client-credentials token refresh buffer and jitter configurableCó thể đã có người làm Có pull request liên kết đang mở hoặc đã được merge. Đang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 72/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 25/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
openfga/java-sdk#370 · 2 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
openfga/java-sdk#361 · 1 reaction ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của openfga/java-sdk
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
-
area/docs
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 88/100
Maintainer thường phản hồi trong vòng 1 ngày
-
BoxChart rejects valid List.of data with NullPointerExceptionCó thể đã có người làm @PHJ2000 đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100