shinrich opened a new pull request #7578: URL: https://github.com/apache/trafficserver/pull/7578
Another issue I encountered with the H2 to origin processing. But this one could affect other code paths as well. In the write_to_net_io(), we deliver the WRITE_READY event to the write_vio continuation for processing. It is possible in the handlers, the write_vio is adjusted so the continuation is changed. In that case, the write processing should be restarted to make the appropriate changes to the new version of the write_vio. In my original version, I just added the continuation check to the proceeding if. So if the write_signal_and_update returned something other than EVENT_CONT or the continuation in the vio changed, we just returned. However, in that case we would lose the WRITE_READY. By rescheduling the WRITE_READY is delivered to the new continuation if there is more data to be processed than handled by the original continuation. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
