Skip to content

[extension/azureauth] Add support to be used in receivers #39012

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

Closed
constanca-m opened this issue Mar 27, 2025 · 1 comment · Fixed by #39033
Closed

[extension/azureauth] Add support to be used in receivers #39012

constanca-m opened this issue Mar 27, 2025 · 1 comment · Fixed by #39033
Labels

Comments

@constanca-m
Copy link
Contributor

constanca-m commented Mar 27, 2025

Component(s)

extension/azureauth

Is your feature request related to a problem? Please describe.

For the extension to have support to be used in receivers, it needs to implement:

  • Interface extensionauth.Server so that it can check if requests have valid authenticaton
  • Interface azcore.TokenCredential so the receivers can get a token if necessary (it will be for all azure receivers).

Describe the solution you'd like

All the examples are in the New component issue: #38908.

Describe alternatives you've considered

No response

Additional context

No response

@constanca-m constanca-m added enhancement New feature or request needs triage New item requiring triage labels Mar 27, 2025
Copy link
Contributor

Pinging code owners:

See Adding Labels via Comments if you do not have permissions to add labels yourself.

@github-actions github-actions bot removed the needs triage New item requiring triage label Mar 27, 2025
@constanca-m constanca-m changed the title [extension/azureauth] Implement extensionauth.Server interface [extension/azureauth] Add support to be used in receivers Mar 28, 2025
atoulme pushed a commit that referenced this issue Mar 30, 2025
<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description

For the extension to have support to be used in receivers, it needs to
implement:

- Interface `extensionauth.Server` so that it can check if requests have
valid authentication.
- Interface `azcore.TokenCredential` so the receivers can get a token if
necessary (it will be for all azure receivers).


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

<!--Describe what testing was performed and which tests were added.-->
#### Testing

I have added unit tests.

<!--Describe the documentation added.-->
#### Documentation

Code comments and README file updated.
dmathieu pushed a commit to dmathieu/opentelemetry-collector-contrib that referenced this issue Apr 8, 2025
…etry#39033)

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

For the extension to have support to be used in receivers, it needs to
implement:

- Interface `extensionauth.Server` so that it can check if requests have
valid authentication.
- Interface `azcore.TokenCredential` so the receivers can get a token if
necessary (it will be for all azure receivers).


<!-- Issue number (e.g. open-telemetry#1234) or full URL to issue, if applicable. -->
#### Link to tracking issue
Fixes open-telemetry#39012.

<!--Describe what testing was performed and which tests were added.-->
#### Testing

I have added unit tests.

<!--Describe the documentation added.-->
#### Documentation

Code comments and README file updated.
Fiery-Fenix pushed a commit to Fiery-Fenix/opentelemetry-collector-contrib that referenced this issue Apr 24, 2025
…etry#39033)

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

For the extension to have support to be used in receivers, it needs to
implement:

- Interface `extensionauth.Server` so that it can check if requests have
valid authentication.
- Interface `azcore.TokenCredential` so the receivers can get a token if
necessary (it will be for all azure receivers).


<!-- Issue number (e.g. open-telemetry#1234) or full URL to issue, if applicable. -->
#### Link to tracking issue
Fixes open-telemetry#39012.

<!--Describe what testing was performed and which tests were added.-->
#### Testing

I have added unit tests.

<!--Describe the documentation added.-->
#### Documentation

Code comments and README file updated.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
1 participant