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

[configauth] Fix unmarshaling of confighttp.AuthConfig (#10749) #10753

Merged
merged 1 commit into from
Jul 30, 2024

Conversation

songy23
Copy link
Member

@songy23 songy23 commented Jul 30, 2024

cherry-pick of #10749

…y#10749)

#### Description

- Uses `mapstructure:",squash"` instead of `mapstructure:"-"` for the
Authenticator field
- Removes pointer (since pointers can't be squashed)
- Adds test in OTLP receiver for this reproducing the user's issue

<!-- Issue number if applicable -->
#### Link to tracking issue
Fixes open-telemetry#10750

---------

Co-authored-by: Pablo Baeyens <pablo.baeyens@datadoghq.com>
@songy23 songy23 requested review from a team and TylerHelmuth and removed request for a team July 30, 2024 15:25
@songy23
Copy link
Member Author

songy23 commented Jul 30, 2024

(skipping CIs & review since this is just a cherry-pick)

@songy23 songy23 merged commit 54c1961 into open-telemetry:release/v0.106.x Jul 30, 2024
23 of 26 checks passed
@songy23 songy23 deleted the release/v0.106.x branch July 30, 2024 15:28
@github-actions github-actions bot added this to the next release milestone Jul 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants