Bug: No telemetry recorded for failed requests
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 48/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 停滞
- 技術スタック
- java
調査の方向性
src/main/java/dev/openfga/sdk/api/client/HttpRequestAttempt.java から始め、211-221 行付近の成功パスでのメトリクス記録と、再試行を使い果たした場合および再試行不可能なエラーの終端失敗パスに注目してください。既存のステータスコードによる区別を維持しつつ、失敗したリクエストと成功したリクエストの両方で requestDuration と queryDuration が記録されるようにしてください。
索引モデルが issue の本文から書いたものです。
説明
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)
- 主要言語
- Java
- スター
- 55
- フォーク
- 26
- 平均マージ
- 1日 22時間
- マージ済み PR(30日)
- 5
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
openfga/java-sdk のほかの issue
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
メンテナーはふだん 1 日以内に返信
-
Make client-credentials token refresh buffer and jitter configurable対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープン
難易度 3/5 1〜2日 初心者へのやさしさ 72/100
メンテナーはふだん 1 日以内に返信
-
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
メンテナーはふだん 1 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
openfga/java-sdk#370 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 5/5 1週間以上 初心者へのやさしさ 35/100
openfga/java-sdk#361 · リアクション 1 件 ·
メンテナーはふだん 1 日以内に返信
openfga/java-sdk の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 83/100
jenkinsci/gitlab-plugin#1950 ·
-
It's not necessary to copy the memory block in the readWrite() of org.h2.store.fs.mem.FileMemDataオープン
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
h2database/h2database#4435 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
micronaut-projects/micronaut-core#13717 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
ADORSYS-GIS/token-status-link#145 ·
メンテナーはふだん 3 日以内に返信
-
enhancement
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
helidon-io/helidon#12721 ·
メンテナーはふだん 1 日以内に返信