fluent / fluent-plugin-kafka

Kafka input and output plugin for Fluentd
Other
303 stars 176 forks source link

allow to have a dedicated key for headers content #498

Open ttych opened 9 months ago

ttych commented 9 months ago

Hi Maintainers !

It is a pull request, to extend a current feature.

It is possible for some inputs to inject kafka message headers into the message (merge). But for some scenario, there can be some overlap between kafka message keys and headers keys, and headers overwrite message content during merge with headers content.

Even the add_headers was not present in the basic kafka input.

So the purpose of this PR is to :

I tried to add some tests, while tests are working fine (with my local kafka instance) with test/plugin/test_in_kafka.rb, tests are flappy with test/plugin/test_in_kafka_group.rb (I don't know if it is related to the kafka settings on my desktop or related to the kafka input processing).

Thanks,

Regards,

ttych commented 9 months ago

Hi,

Don't hesitate to comment, amend, reject the PR ;) My motivation behind this is to be able to split and identify what comes from the message "body" and what comes from the message "header". Thanks.

github-actions[bot] commented 1 month ago

This PR has been automatically marked as stale because it has been open 90 days with no activity. Remove stale label or comment or this PR will be closed in 30 days