[trafficserver] branch master updated: Go into timeout state regardless of whether server response data was sent.

[sc...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

scw00 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/master by this push:
     new 60b2d99  Go into timeout state regardless of whether server response data was sent.
60b2d99 is described below

commit 60b2d998283e0c05e1270bd1870303f47c52949e
Author: Susan Hinrichs <sh...@apache.org>
AuthorDate: Mon Mar 19 10:26:01 2018 -0500

    Go into timeout state regardless of whether server response data was sent.
    
    We had issue where client timed out in the POST tunnel state but client_response_hdr_bytes
    was non-zero, so the original post tunnel was not shutdown.  Causing an assert in consumer_handler later.
    
    client_response_hdr_bytes may be set in the setup_100_continue logic, cache logic, and probably a few other
    paths in addition to the receive response header logic.
---
 proxy/http/HttpSM.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/proxy/http/HttpSM.cc b/proxy/http/HttpSM.cc
index 8d53c83..1461866 100644
--- a/proxy/http/HttpSM.cc
+++ b/proxy/http/HttpSM.cc
@@ -2653,7 +2653,7 @@ HttpSM::tunnel_handler_post(int event, void *data)
 
   switch (event) {
   case HTTP_TUNNEL_EVENT_DONE: // Tunnel done.
-    if (p->handler_state == HTTP_SM_POST_UA_FAIL && client_response_hdr_bytes == 0) {
+    if (p->handler_state == HTTP_SM_POST_UA_FAIL) {
       // post failed
       switch (t_state.client_info.state) {
       case HttpTransact::ACTIVE_TIMEOUT:

-- 
To stop receiving notification emails like this one, please contact
scw00@apache.org.