shinrich opened a new pull request #8201: URL: https://github.com/apache/trafficserver/pull/8201
Noticed this while debugging HTTP2 to origin. stream->update_sent_count() updates the write_vio.ndone. But in the original code this update is done after the stream->is_write_vio_done() check is made to determine whether the EOS flag should be set. The EOS flag will only be set if the payload is 0. I would think this would have caused problems quite a lot of the time. Generally showing up as timeouts as the client waits forever for the EOS byte after receiving all the data. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
