Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

github: add CIFuzz workflow #374

Closed
wants to merge 1 commit into from

Conversation

DavidKorczynski
Copy link
Contributor

@DavidKorczynski DavidKorczynski commented Nov 23, 2022

Add CIFuzz workflow action to have fuzzers build and run on each PR.

This is a service offered by OSS-Fuzz where LibHTP already runs. CIFuzz can help detect regressions and catch fuzzing build issues early, and has a variety of features (see the URL above). In the current PR the fuzzers gets build on a pull request and will run for 300 seconds.

@catenacyber do you think this would be helpful here?

@catenacyber
Copy link
Contributor

Hi David, I am not so sure because we are in the process of moving this to a rust crate that is https://github.com/cccs-rtmorti/libhtp-rs

I hope this repository has not long to live until and will not see many PRs.

Anyways, that should not hurt too much anyways...

I plan to integrate the differential fuzzing between both version cf https://github.com/catenacyber/oss-fuzz/tree/libhtp-diff ;-)

@DavidKorczynski
Copy link
Contributor Author

Okay, sounds good @catenacyber -- I'll leave it open in case you want to have it running for the remainder of the project's lifetime!

@catenacyber
Copy link
Contributor

Thanks David

Out of curiosity, how did you end up here ?

@DavidKorczynski
Copy link
Contributor Author

Out of curiosity, how did you end up here ?

Nothing specific, simply just out here trying to help :)

@victorjulien
Copy link
Member

Merged, thanks a lot @DavidKorczynski !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants