search

djc / askama

Type-safe, compiled Jinja-like templates for Rust
Apache License 2.0
3.43k stars 218 forks source link

fuzz: Add cifuzz integration #1028

Closed manunio closed 5 months ago

manunio commented 5 months ago

Add cifuzz workflow action to have fuzzers build and run on each PR. This is a service offered by oss-fuzz where askama already runs. cifuzz can help catch shallow bugs, regressions and fuzzing build issues before they are merged into the repository, to do this fuzzing will be run for ~5min.

djc commented 5 months ago

Thanks! 5 minutes feels kind of long when the complete CI workflow right now completes in 3.5. Can we make it like 3 minutes, or do we lose a lot of that to setup stuff?

manunio commented 5 months ago

Thanks! 5 minutes feels kind of long when the complete CI workflow right now completes in 3.5. Can we make it like 3 minutes, or do we lose a lot of that to setup stuff?

I think 3 minutes is fine for now, as setup time is not inclusive of fuzzing seconds, it will most probably take somewhere around 3 to 4 mins for each run.