Add fuzz harnesses, sanitizer options, and CI workflows; fix Transfer-Encoding normalization in tests#54
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 225baa3922
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: a4896c5374
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: b032b6602b
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Motivation
chunkedand add regression cases to prevent regressions.Description
transaction_fuzzerandrule_parser_fuzzerwith stdin drivers, supporting corpora and atest/fuzzer/README.md, and wired them intotest/fuzzer/Makefile.amso they are built with the test harnesses.fuzz-smoke.ymlfor PR smoke runs,nightly-fuzz.ymlfor scheduled mutation sweeps, andsanitizers.ymlfor ASan/UBSan builds that runmakeandmake check.configure.acto add--enable-asan,--enable-ubsan, and--enable-tsanflags and to refine AFL/coverage flags for clang, and to propagate sanitizer CFLAGS/LDFLAGS into the build.test/regression/regression_test.ccby addingtrim_asciiand updatinghas_chunked_headerto parse comma-separated Transfer-Encoding token lists; added regression tests intest/test-cases/regression/request-transfer-encoding-normalization.json.Testing
make -jandmake checkas part of thesanitizers.ymlworkflow and completed without failures in the observed run.make -C test/fuzzerand exercises them with seed corpus runs and randomized loops as defined infuzz-smoke.yml, with no errors observed.nightly-fuzz.ymland completed the scheduled run without errors.Codex Task