Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Functions for modifying a Composite Propagator #954

Open
alanwest opened this issue Sep 15, 2020 · 0 comments
Open

Functions for modifying a Composite Propagator #954

alanwest opened this issue Sep 15, 2020 · 0 comments
Labels
release:after-ga Not required before GA release, and not going to work on before GA spec:context Related to the specification/context directory

Comments

@alanwest
Copy link
Member

The ability to create a composite propagator is specified. In .NET, instrumentation defaults to use a composite propagator that supports propagating W3C trace context and baggage.

Currently, if a user wants to enable an additional propagator (e.g., Zipkin's B3 format), but also maintain the W3C trace context and baggage propagators, they would need to create a new composite propagator and make sure they add all three.

I'd like to suggest adding a means to modify an existing composite propagator. Modifications like adding to, removing from, and perhaps clearing the list of propagators. It may also be nice to have "contains" functionality to check if a composite propagator contains a certain type of propagator.

@alanwest alanwest added the spec:context Related to the specification/context directory label Sep 15, 2020
@andrewhsu andrewhsu added the release:after-ga Not required before GA release, and not going to work on before GA label Sep 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release:after-ga Not required before GA release, and not going to work on before GA spec:context Related to the specification/context directory
Projects
None yet
Development

No branches or pull requests

2 participants