Author: zwoop
Date: Sun Jan 8 20:35:28 2012
New Revision: 1228947
URL: http://svn.apache.org/viewvc?rev=1228947&view=rev
Log:
TS-1056 Lost UA connections can show up as "400 ERR_INVALID_REQ" in logs
Modified:
trafficserver/traffic/trunk/CHANGES
trafficserver/traffic/trunk/proxy/http/HttpSM.cc
Modified: trafficserver/traffic/trunk/CHANGES
URL:
http://svn.apache.org/viewvc/trafficserver/traffic/trunk/CHANGES?rev=1228947&r1=1228946&r2=1228947&view=diff
==============================================================================
--- trafficserver/traffic/trunk/CHANGES (original)
+++ trafficserver/traffic/trunk/CHANGES Sun Jan 8 20:35:28 2012
@@ -1,5 +1,8 @@
-*- coding: utf-8 -*-
Changes with Apache Traffic Server 3.1.2
+ *) [TS-1056] Lost UA connections can show up as "400 ERR_INVALID_REQ"
+ in logs.
+
*) [TS-1048] Add TS API to enable plugins to use traffic server
configuration infrastructure. Author: Bianca Cooper.
Modified: trafficserver/traffic/trunk/proxy/http/HttpSM.cc
URL:
http://svn.apache.org/viewvc/trafficserver/traffic/trunk/proxy/http/HttpSM.cc?rev=1228947&r1=1228946&r2=1228947&view=diff
==============================================================================
--- trafficserver/traffic/trunk/proxy/http/HttpSM.cc (original)
+++ trafficserver/traffic/trunk/proxy/http/HttpSM.cc Sun Jan 8 20:35:28 2012
@@ -669,16 +669,17 @@ HttpSM::state_read_client_request_header
int bytes_used = 0;
ink_assert(ua_entry->eos == false);
-
switch (event) {
- case VC_EVENT_EOS:
- ua_entry->eos = true;
- // Fall through
case VC_EVENT_READ_READY:
case VC_EVENT_READ_COMPLETE:
// More data to parse
break;
+ case VC_EVENT_EOS:
+ ua_entry->eos = true;
+ if (client_request_hdr_bytes !=0)
+ break;
+ // Fall through
case VC_EVENT_ERROR:
case VC_EVENT_INACTIVITY_TIMEOUT:
case VC_EVENT_ACTIVE_TIMEOUT:
