Skip to content

Conversation

@khushijain21
Copy link
Contributor

@khushijain21 khushijain21 commented Aug 4, 2025

What does this PR do?

This PR makes the logger on TLSConfig global - because it is required to be able to set this field even outside this package. One such example is in beatsauthextension where we directly configure TLSConfig

https://github.com/elastic/opentelemetry-collector-components/blob/main/extension/beatsauthextension/config.go#L28

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have added tests that prove my fix is effective or that my feature works

@khushijain21 khushijain21 requested a review from a team as a code owner August 4, 2025 09:02
@khushijain21 khushijain21 requested review from VihasMakwana and faec and removed request for a team August 4, 2025 09:02
VihasMakwana
VihasMakwana previously approved these changes Aug 4, 2025
@khushijain21 khushijain21 enabled auto-merge (squash) August 4, 2025 09:22
@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

History

@khushijain21 khushijain21 merged commit 714a3cd into elastic:main Aug 4, 2025
5 checks passed
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.

3 participants