HttpTraceContextPropagator should not have "HTTP" in the name

[aabmass]

The W3C TraceContext propagator is called HttpTraceContextPropagator. I don't think the "HTTP" prefix should be in the name, since propagation is carrier-agnostic. Other OTel SDK names:

• In Java, it's W3CTraceContextPropagator
• In Python, TraceContextTextMapPropagator

[dyladan]

Discussed in SIG. We like TraceContextTextMapPropagator

[aabmass]

@dyladan I just realized we have the same problem with the W3C Baggage propagator, which is called HttpBaggagePropagator

opentelemetry-js/packages/opentelemetry-core/src/baggage/propagation/HttpBaggagePropagator.ts

Line 45 in 30733df

export class HttpBaggagePropagator implements TextMapPropagator {

• Python has W3CBaggagePropagator
• Java also has W3CBaggagePropagator

I would guess W3CBaggagePropagator makes the most sense here.. But then we have TraceContextTextMapPropagator without "W3C" in it. Thoughts?

[dyladan]

I think i'd rather have W3C in both of them. W3CTraceContextTextMapPropagator (a mouthful I know) and W3CBaggagePropagator.

@open-telemetry/javascript-maintainers thoughts?

[aabmass]

One inconsistency there is the -TextMap- part. I think either both or neither should have it (they both implement TextMapPropagator)

[dyladan]

W3CTraceContextTextMapPropagator and W3CBaggageTextMapPropagator ?

[obecny]

would be in favour of making them shorter W3CTraceContextPropagator and W3CBaggagePropagator ?

[dyladan]

Sounds good to me