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

[receiver/splunkhec] support different splitting strategies #22799

Conversation

atoulme
Copy link
Contributor

@atoulme atoulme commented May 26, 2023

Description:
Support different strategies for splitting payloads when receiving a request with the Splunk HEC receiver.

Link to tracking Issue:
Fixes #22788

Testing:
Added tests for the functionality

Documentation:
Added to README.

@atoulme atoulme requested review from a team and fatsheep9146 May 26, 2023 00:02
@atoulme atoulme force-pushed the support_different_splitting_strategies_hec_raw branch from 2986a68 to 402b58c Compare May 26, 2023 00:05
@atoulme atoulme force-pushed the support_different_splitting_strategies_hec_raw branch from 402b58c to 678f485 Compare May 26, 2023 05:31
Copy link
Contributor

@splunkericl splunkericl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for the quick implementation

Copy link
Member

@dmitryax dmitryax left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGMT, just one comment

receiver/splunkhecreceiver/config.go Outdated Show resolved Hide resolved
Copy link
Member

@dmitryax dmitryax left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGMT, just one comment

atoulme and others added 2 commits May 30, 2023 11:39
Co-authored-by: Dmitrii Anoshin <anoshindx@gmail.com>
@dmitryax dmitryax merged commit 55f1387 into open-telemetry:main May 30, 2023
@github-actions github-actions bot added this to the next release milestone May 30, 2023
Caleb-Hurshman pushed a commit to observIQ/opentelemetry-collector-contrib that referenced this pull request Jul 6, 2023
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.

[SplunkHecReceiver] Allows users to disable line breaking for Raw endpoint
5 participants