search

SeisoLLC / zeek-kafka

A Zeek log writer plugin that publishes to Kafka.
Apache License 2.0
45 stars 16 forks source link

Update librdkafka? #32

Open chuck-alt-delete opened 2 years ago

chuck-alt-delete commented 2 years ago

The version of librdkafka seems fixed at 1.4.2, which is about a year and a half old at this point. Any reason not to use a newer version? Maybe this can be incorporated into CI?

Also, all the examples show compiling librdkafka from source. Any reason not to install it with a package manager?

JonZeolla commented 2 years ago

@chuck-alt-delete yeah that shouldn't be a problem. To run the tests with a new version the only update is to https://github.com/SeisoLLC/zeek-kafka/blob/main/docker/docker-compose.yml#L49

chuck-alt-delete commented 2 years ago

Thanks Jon!

Sort of related is:

If docker image artifacts are published somewhere, maybe it makes sense to have images with different versions of zeek and different versions of librdkafka so people can pull off the shelf a trusted, well-tested image with the version combo they need?