getsentry / getsentry/sentry-java

No longer attach outgoing headers in OTel propagators if there's no URL

オープン
#5,725 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る
Improvement Java Platform: Java Spans
主要言語
Kotlin
スター
1.4k
フォーク
478
平均マージ
2日 23時間
マージ済み PR(30日)
67

説明

### Description

Currently we look for a URL on outgoing requests to check against `tracePropagationTargets`. If none can be found in span attributes, we still attach headers. We want to change this to only attach headers if the URL can be found.

This is a breaking change and should land in the next major version.

We also need to check whether there's any cases where there's no URL attributes but the SDK should still attach headers, e.g. for Queue producers and similar things.

Once checked also make sure to update (dev) docs.

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

Inspect the OpenTelemetry propagator handling for outgoing requests and how tracePropagationTargets is applied when no URL attribute exists. Check queue producers and similar cases for valid URL-less propagation, then update the developer documentation for the breaking change. Done means the intended propagation behavior and documented exceptions are verified.

索引モデルが issue の本文から書いたものです。

評価

技術スタック
kotlin
領域
observability-sre
issue の種類
機能追加
難易度
4/5
見積もり時間
3〜5日
活発さ
静か
明瞭さ
おおむね明確
初心者へのやさしさ
45/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。