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

[chore] Manual implementation of extensionauth.Client interface #38451

Merged
merged 5 commits into from
Mar 12, 2025

Conversation

mx-psi
Copy link
Member

@mx-psi mx-psi commented Mar 7, 2025

Description

Prepares for addressing changes from open-telemetry/opentelemetry-collector#12574

github-merge-queue bot pushed a commit to open-telemetry/opentelemetry-collector that referenced this pull request Mar 12, 2025
<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description

An attempt at splitting `extensionauth.Client` by protocol type.
open-telemetry/opentelemetry-collector-contrib/pull/38451 removes usages
of the `NewClient` and `NewServer` constructor

---------

Co-authored-by: Jade Guiton <[email protected]>
@mx-psi mx-psi marked this pull request as ready for review March 12, 2025 12:41
@mx-psi mx-psi requested a review from a team as a code owner March 12, 2025 12:41
@mx-psi mx-psi merged commit 58c9f6f into open-telemetry:main Mar 12, 2025
171 checks passed
@github-actions github-actions bot added this to the next release milestone Mar 12, 2025
mx-psi added a commit that referenced this pull request Mar 13, 2025
…mentions (#38601)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description

<!-- Issue number (e.g. #1234) or full URL to issue, if applicable. -->

Follows #38451 and open-telemetry/opentelemetry-collector/pull/12574.

Removes explicit mentions to `extensionauth.Client` and replaces them by
`extensionauth.HTTPClient` and `extensionauth.GRPCClient`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants