You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by sh...@apache.org on 2015/06/09 15:11:15 UTC

trafficserver git commit: TS-3640: Drupal auth fails over SPDY

Repository: trafficserver
Updated Branches:
  refs/heads/master 3b097aff4 -> c8bcd7006


TS-3640: Drupal auth fails over SPDY


Project: http://git-wip-us.apache.org/repos/asf/trafficserver/repo
Commit: http://git-wip-us.apache.org/repos/asf/trafficserver/commit/c8bcd700
Tree: http://git-wip-us.apache.org/repos/asf/trafficserver/tree/c8bcd700
Diff: http://git-wip-us.apache.org/repos/asf/trafficserver/diff/c8bcd700

Branch: refs/heads/master
Commit: c8bcd7006ffe66e4dd2d8f072da771cd947e4643
Parents: 3b097af
Author: shinrich <sh...@yahoo-inc.com>
Authored: Tue Jun 9 08:10:41 2015 -0500
Committer: shinrich <sh...@yahoo-inc.com>
Committed: Tue Jun 9 08:10:41 2015 -0500

----------------------------------------------------------------------
 proxy/http/HttpSM.cc | 15 +++------------
 1 file changed, 3 insertions(+), 12 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/trafficserver/blob/c8bcd700/proxy/http/HttpSM.cc
----------------------------------------------------------------------
diff --git a/proxy/http/HttpSM.cc b/proxy/http/HttpSM.cc
index 91c1fda..801e263 100644
--- a/proxy/http/HttpSM.cc
+++ b/proxy/http/HttpSM.cc
@@ -3151,18 +3151,9 @@ HttpSM::tunnel_handler_ua(int event, HttpTunnelConsumer *c)
 
   ink_assert(ua_entry->vc == c->vc);
   if (close_connection) {
-    // If the client could be pipelining or is doing a POST, we need to
-    //   set the ua_session into half close mode
-
-    // only external POSTs should be subject to this logic; ruling out internal POSTs here
-    bool is_eligible_post_request = (t_state.method == HTTP_WKSIDX_POST);
-    if (is_eligible_post_request) {
-      NetVConnection *vc = ua_session->get_netvc();
-      if (vc) {
-        is_eligible_post_request = vc->get_is_internal_request() ? false : true;
-      }
-    }
-    if ((is_eligible_post_request || t_state.client_info.pipeline_possible == true) && c->producer->vc_type != HT_STATIC &&
+    // If the client could be pipelining, we need to set the ua_session into half close mode
+
+    if (t_state.client_info.pipeline_possible == true && c->producer->vc_type != HT_STATIC &&
         event == VC_EVENT_WRITE_COMPLETE) {
       ua_session->set_half_close_flag();
     }