Add fuzzing to CI#12887
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #12887 +/- ##
=======================================
Coverage 98.95% 98.95%
=======================================
Files 131 131
Lines 47800 47886 +86
Branches 2480 2492 +12
=======================================
+ Hits 47300 47386 +86
Misses 376 376
Partials 124 124
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. |
Merging this PR will not alter performance
Comparing Footnotes
|
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
| from aiohttp.http_exceptions import BadHttpMessage | ||
| from aiohttp.http_parser import HttpRequestParserC, HttpRequestParserPy | ||
|
|
||
| AIOHTTP_VAL = 0 |
There was a problem hiding this comment.
I'm unclear how this variable is supposed to change.
| FuzzStream(fdp.ConsumeBytes(atheris.ALL_REMAINING)), | ||
| ) | ||
| if not obj.at_eof(): | ||
| await obj.form() |
There was a problem hiding this comment.
Can we get the coverage from the fuzzing run? I'm not sure this is covering enough of the multipart code.
Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com> Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>
Co-authored-by: Sam Bull <aa6bs0@sambull.org>
Co-authored-by: Sam Bull <aa6bs0@sambull.org>
|
@DavidKorczynski Could you check over this PR and confirm how to complete the integration with oss-fuzz repo? |
No description provided.