Skip to content

Conversation

@khushijain21
Copy link
Contributor

@khushijain21 khushijain21 commented Sep 1, 2025

Beatsauth extension now replaces the incoming round-tripper with beat's implementation of it.
The input to beatsauthextension is the entire http config block supported by beats - > Ref

How to test this PR

make genelasticcol

Create config.yaml and use the below config

receivers:
 filelog:
   include_file_name: true
   include:
     - "./otlp-all.json"

extensions:
 beatsauth:
   ssl:
    enabled: true
    verification_mode: none
   timeout: 9s
   
exporters:
 debug:
   verbosity: detailed
 elasticsearch:
  endpoints:
   -  https://localhost:9200
  password: testing
  user: admin
  auth:
   authenticator: beatsauth

service:
 extensions: [beatsauth]
 pipelines:
   logs:
     receivers: [filelog]
     processors: []
     exporters: [debug, elasticsearch]
./_build/elastic-collector-components --config config.yml

Relates elastic/elastic-agent#9633

@khushijain21 khushijain21 requested a review from a team as a code owner September 1, 2025 09:06
@khushijain21 khushijain21 marked this pull request as draft September 1, 2025 09:07
@khushijain21 khushijain21 changed the title use http settings on beatsauth Use HTTP settings on beatsauth extension Sep 2, 2025
@khushijain21 khushijain21 marked this pull request as ready for review September 2, 2025 04:19
khushijain21 and others added 3 commits September 4, 2025 11:48
Co-authored-by: Tiago Queiroz <me@tiago.life>
Co-authored-by: Mauri de Souza Meneguzzo <mauri870@gmail.com>
@cmacknz
Copy link
Member

cmacknz commented Sep 5, 2025

Besides the cleanup of getHTTPOptions to make it clearer what we are omitting and why in the comments, this LGTM.

Co-authored-by: Craig MacKenzie <craig.mackenzie@elastic.co>
@khushijain21 khushijain21 merged commit 386ed02 into elastic:main Sep 9, 2025
12 of 13 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.

4 participants