getsentry / getsentry/sentry-java

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

Offen
#5,725 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Improvement Java Platform: Java Spans
Vorherrschende Sprache
Kotlin
Sterne
1.4k
Forks
478
Ø Merge
2 T. 23 Std.
Gemergte PRs (30 T.)
67

Beschreibung

### 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.

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

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.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
kotlin
Bereich
observability-sre
Issue-Typ
Feature
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Ruhig
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.