shinrich commented on pull request #8201: URL: https://github.com/apache/trafficserver/pull/8201#issuecomment-908393329
Removed the !resp_reader->is_read_avail_more_than(0) test that was guarding the setting of the EOS bit. Either the write_vio nbytes is set to a specific nbytes and the payload_length got us there. Or it didn't . The read_avail will still be > 0 until we get to the xmit() call and that is ok. Should still set the EOS flag, if the data still in the buffer is the last set we will send. Ran it in my production test with a Warning message for the EOS and payload > 0 case. We do run into that case. The overall numbers look fine. Ran a few of the h2 autests and they seem fine. Will see how the PR autest goes.. -- 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]
