Decide whether we want to check `tracePropagationTargets` before attaching tracing headers in OpenTelemetry propagator

[adinauer]

Currently Java SDK does not check the target URL against tracePropagationTargets before attaching sentry-trace and baggage headers to an outgoing request.

The URL may have been attached to the span attributes but it might not be there.

Do we want to check before attaching these headers?
Also keep in mind other SDKs that have POTel.

[cleptric]

@adinauer was this always the case in Java or just a new behavior due to Potel?

[adinauer]

@cleptric it's always been that way since we introduced our first propagator 2+ years ago.

[cleptric]

Are we only talking about Otel or tracing in general?

[adinauer]

OpenTelemetry only, the rest of the SDK does check the URL where possible (not all libraries allow us to get the URL).

[cleptric]

We should take tracePropagationTargets into account for our SDK OTel propagators.
E.g. JS does https://github.com/getsentry/sentry-javascript/blob/develop/packages/opentelemetry/src/propagator.ts#L131-L151.

[adinauer]

Thanks for the clarification. Will change it in Java SDK.

[adinauer]

Java SDK now checks tracePropagationTargets in OTel propagator (getsentry/sentry-java#4191)