[trafficserver] branch 8.0.x updated: Revert "Two more places to check whether attempting half_closed connection logic is feasible."

[zwoop]

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

zwoop pushed a commit to branch 8.0.x
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/8.0.x by this push:
     new 6ac3b2a  Revert "Two more places to check whether attempting 
half_closed connection logic is feasible."
6ac3b2a is described below

commit 6ac3b2ad39b9ca0c3fcb421a582978ca2a3c63aa
Author: Susan Hinrichs <shinr...@oath.com>
AuthorDate: Tue Dec 18 16:00:40 2018 +0000

    Revert "Two more places to check whether attempting half_closed connection 
logic is feasible."
    
    This reverts commit 653927f9aab9a3cbe8f09521dd8c8154ccbc1614.
    
    Erroneous fix caused KEEPALIVE to fail on HTTP/1.1 over HTTPS
    
    (cherry picked from commit f6fe5f0079005813d33d1cddd9d127c75a1f5e0b)
---
 proxy/ProxyClientSession.cc | 3 +--
 proxy/http/HttpSM.cc        | 7 +++----
 2 files changed, 4 insertions(+), 6 deletions(-)

diff --git a/proxy/ProxyClientSession.cc b/proxy/ProxyClientSession.cc
index 2f80f3b..8cb61eb 100644
--- a/proxy/ProxyClientSession.cc
+++ b/proxy/ProxyClientSession.cc
@@ -206,8 +206,7 @@ ProxyClientSession::handle_api_return(int event)
     break;
   }
   default:
-    Error("received invalid session hook %s (%d)", 
HttpDebugNames::get_api_hook_name(hookid), hookid);
-    ink_release_assert(false);
+    Fatal("received invalid session hook %s (%d)", 
HttpDebugNames::get_api_hook_name(hookid), hookid);
     break;
   }
 }
diff --git a/proxy/http/HttpSM.cc b/proxy/http/HttpSM.cc
index 4131d45..a0a48e8 100644
--- a/proxy/http/HttpSM.cc
+++ b/proxy/http/HttpSM.cc
@@ -3176,8 +3176,7 @@ HttpSM::tunnel_handler_ua(int event, HttpTunnelConsumer 
*c)
     c->write_success          = true;
     t_state.client_info.abort = HttpTransact::DIDNOT_ABORT;
     if (t_state.client_info.keep_alive == HTTP_KEEPALIVE) {
-      if (ua_txn->allow_half_open() &&
-          (t_state.www_auth_content != HttpTransact::CACHE_AUTH_SERVE || 
ua_txn->get_server_session())) {
+      if (t_state.www_auth_content != HttpTransact::CACHE_AUTH_SERVE || 
ua_txn->get_server_session()) {
         // successful keep-alive
         close_connection = false;
       }
@@ -3238,8 +3237,8 @@ HttpSM::tunnel_handler_ua(int event, HttpTunnelConsumer 
*c)
         is_eligible_post_request &= !vc->get_is_internal_request();
       }
     }
-    if ((is_eligible_post_request || t_state.client_info.pipeline_possible == 
true) && ua_txn->allow_half_open() &&
-        c->producer->vc_type != HT_STATIC && event == VC_EVENT_WRITE_COMPLETE) 
{
+    if ((is_eligible_post_request || t_state.client_info.pipeline_possible == 
true) && c->producer->vc_type != HT_STATIC &&
+        event == VC_EVENT_WRITE_COMPLETE) {
       ua_txn->set_half_close_flag(true);
     }