shinrich opened a new pull request #8301:
URL: https://github.com/apache/trafficserver/pull/8301


   I have noticed occasional crashes in working with the H2 to origin branch 
and in our 9.1 with producer_run().  Both branches include PR #7976.  With this 
change, we need to be ready to respond to both read and write events on the 
server_entry->vc (read for responses and write for post bodies).  Currently we 
do a vio check in both state_read_server_response_header and 
state_send_server_request_header to hand the incorrectly passed read/write to 
othe other side.
   
   In general with the vc_entry we have parallel entries for read and write 
operations, except for the vc_handler.  This does result in some write events 
unexpectedly showing up in a read handler and visa versa which ends up with 
confusing code.
   
   This PR, splits the vc_handler into vc_read_handler and vc_write_handler.  
That should hopefully simplify some of handler logic moving forward.


-- 
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]


Reply via email to