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

[configgrpc] IncludeMetadata is experimental #9642

Closed
TylerHelmuth opened this issue Feb 26, 2024 · 0 comments · Fixed by #10685
Closed

[configgrpc] IncludeMetadata is experimental #9642

TylerHelmuth opened this issue Feb 26, 2024 · 0 comments · Fixed by #10685
Labels
area:config good first issue Good for newcomers release:required-for-ga Must be resolved before GA release

Comments

@TylerHelmuth
Copy link
Member

TylerHelmuth commented Feb 26, 2024

From configgrpc.go

// Include propagates the incoming connection's metadata to downstream consumers.
// Experimental: *NOTE* this option is subject to change or removal in the future.
IncludeMetadata bool `mapstructure:"include_metadata"`

This issue tracks any work needed specifically for configgrpc, but the problem spans both confighttp and configgrpc in similar ways. For the sake of centralized discussion, lets start in #9381 and move here only if there are configgrpc-specific concerns.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:config good first issue Good for newcomers release:required-for-ga Must be resolved before GA release
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants