Skip to content

Conversation

pwhelan
Copy link
Contributor

@pwhelan pwhelan commented Sep 25, 2025

Summary

The in_splunk plugin is responding with {"text": "Invalid data format"} even when submitting valid payloads.


Fluent Bit is licensed under Apache 2.0, by submitting this pull request I understand that this code will be released under the terms of that license.

Copy link

coderabbitai bot commented Sep 25, 2025

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch pwhelan-fix-in-splunk-invalid-data-format-http1

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@cosmo0920 cosmo0920 self-requested a review September 26, 2025 11:00
Copy link
Contributor

@cosmo0920 cosmo0920 left a comment

Choose a reason for hiding this comment

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

Ah, we need to resolve CI Fuzz task failure.

@pwhelan
Copy link
Contributor Author

pwhelan commented Sep 26, 2025

Ah, we need to resolve CI Fuzz task failure.

The bug seems to be a memory leak when encountering an extremely hard to reproduce parsing error for CLIST or SLIST options. Despite that I did manage to reproduce it artificially and have a fix for it. I am opening a separate set of PRs for that fix.

@cosmo0920
Copy link
Contributor

Got it. Then we need to handle them at first.

@pwhelan
Copy link
Contributor Author

pwhelan commented Sep 26, 2025

Got it. Then we need to handle them at first.

I've done the best I can here:

The memory leaks in the fuzzers seem to happen sporadically, I left some comments about it in #10944.

@cosmo0920 cosmo0920 merged commit 0e09599 into 4.0 Sep 30, 2025
53 of 54 checks passed
@cosmo0920 cosmo0920 deleted the pwhelan-fix-in-splunk-invalid-data-format-http1 branch September 30, 2025 03:50
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.

2 participants