Bruno Garcia

Concept ACK

I couldn't reproduce on MacOS 14.3 with that input file. > If I add limits (1024 for example) to ConsumeRandomLengthByteVector and ConsumeRandomLengthString the harness works. But that could be defeating...

Concept ACK

> On the fuzz ./src/test/fuzz/test http_request.cpp, the issue is that the test itself is passing replySent as true, so the WriteReply() call on the httpserver constructor when the uri_parse is...

> (fwiw I think the bigger optimization you are making here is using This doesn't seem like a big win. Have you benchmarked this to see if it actually makes...

Force-pushed addressing https://github.com/bitcoin/bitcoin/pull/27319#discussion_r1338941982. Also updated the description.

Concept ACK I had this issue when writing #28869.

Concept ACK

Concept ACK

cc: @andrewtoth (https://github.com/bitcoin/bitcoin/pull/26822#pullrequestreview-1238269263)