Add option to capture Redis command arguments in span resource name
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 45/100
- issue の種類
- 機能追加
- 明瞭さ
- おおむね明確
- 活発さ
- 停滞
- 技術スタック
- java, redis
調査の方向性
まず、参照されている dd-trace-rb quantize.rb と IgniteCacheDecorator.java の例を読み、次に dd-trace-java 内の Redis instrumentation を見つけます。設定の動作を定義し、Redis span の resource name に、指定された難読化および切り詰めの上限に従った安全なコマンド引数が含まれることを検証します。
索引モデルが issue の本文から書いたものです。
説明
Library Name
No response
Library Version(s)
No response
Describe the feature you'd like
Add a configuration option to include Redis command arguments (keys) in the span resource name.
Instead of current:
Resource Name: GET
you'd get:
Resource Name: GET user:12345:profile
This feature already exists in dd-trace-rb with DD_REDIS_COMMAND_ARGS, so i don't see a reason why it shouldn't exist here, security reasons are also handled since they do these:
- AUTH commands obfuscated to
AUTH ? - Values truncated to 50 chars
- Full command truncated to 500 chars
- Binary/non-UTF8 data replaced with
?
Reference: dd-trace-rb quantize.rb
Is your feature request related to a problem?
When debugging redis spans, current instrumentation only shows the command type, which makes it very difficult to identify the keys causing the latency.
Describe alternatives you've considered
A similar pattern already exists in dd-trace-java for Ignite cache:
- Config:
DD_IGNITE_CACHE_INCLUDE_KEYS - Implementation:
IgniteCacheDecorator.java#L139-L141
Additional context
No response
- 主要言語
- Java
- スター
- 737
- フォーク
- 361
- 平均マージ
- 3日 20時間
- マージ済み PR(30日)
- 173
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
DataDog/dd-trace-java のほかの issue
-
type: feature request
難易度 1/5 1〜3時間 初心者へのやさしさ 70/100
DataDog/dd-trace-java#10245 · コメント 1 件 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 62/100
DataDog/dd-trace-java#12608 ·
-
type: bug report
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
DataDog/dd-trace-java#12597 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
DataDog/dd-trace-java#12540 · コメント 4 件 · 担当者 1 名 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 25/100
DataDog/dd-trace-java#12480 ·
DataDog/dd-trace-java の issue をすべて見る
似ている issue
-
documentation
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
inu-appcenter/memorIN-backend#288 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
-
frontend maui-pilot pilot-ask question
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
-
area/plugin
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
kestra-io/plugin-kestra#190 ·