search

awslabs / flowgger

A fast data collector in Rust
BSD 2-Clause "Simplified" License
834 stars 55 forks source link

Add fuzzing test to flowgger #79

Closed cahakgeorge closed 6 months ago

cahakgeorge commented 6 months ago

Description of changes: This change seeks to add fuzzing tests to Flowgger. With arbitrary strings as inputs, Flowgger is expected to either throw an error or successfully parse the log entry.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

vche commented 6 months ago

Nice improvements since the first commit !

vche commented 6 months ago

Build fails due to workflow and cross compilation being broken. I've enabled manual runs for the workflow and re ran it on the current release (which previously passed) https://github.com/awslabs/flowgger/actions/runs/9127101215.

And it fails the same way. Since we're using a cross build and openssl is not included anymore, we were using an older version. we'll have to fix this. I've opened an issue for us to fix the workflow - again -.

In the meantime, having tested this locally, and no production code being modified, it is safe to merge.