Doug Patti
Doug Patti
@seliopou I pushed what I think is a more correct implementation for the async library. If you read the commit message, you'll see it has a buffering issue that I...
There's one issue with the two runtime implementations: we can potentially pull more bytes off the socket than we need to parse the headers, at which point you drop those...
Just needs the fixes to the runtimes (or assertions saying it's a known issue) and we're good to merge.
To be clear, the "timeout" here came from a runtime, though in principal it can be substituted with any arbitrary `Client_connection.report_exn`.
Rebased to master
I think I'm seeing a problem here. Consider a change like this: ```diff diff --git a/lib_test/test_server_connection.ml b/lib_test/test_server_connection.ml index d47c62a..f55af7a 100644 --- a/lib_test/test_server_connection.ml +++ b/lib_test/test_server_connection.ml @@ -913,6 +913,8 @@ let test_can_read_more_requests_after_write_eof...
Hey @andreas, this has been on our minds as well recently. My plan was to spend some time working on this this weekend, and I'll definitely take a look at...
> In #148 we made sure the close and flush the body so that the receiver wouldn't potentially be left waiting I actually don't remember if this is true. I...
> 1. might affect her state machine I'm not sure what you mean by this – the httpaf state machine, or the user's state machine? Like if they try to...
Similar discussion on #135. Unprocessable Entity comes from https://tools.ietf.org/html/rfc4918#section-11