[trafficserver] branch master updated: Reverts the changes from TS-3054

[zw...@apache.org]

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

zwoop 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 d372332  Reverts the changes from TS-3054
d372332 is described below

commit d372332b5a8f4cf0c5e5fffa9f4c6c143a53e242
Author: Leif Hedstrom <zw...@apache.org>
AuthorDate: Tue Nov 28 11:17:22 2017 -0700

    Reverts the changes from TS-3054
    
    Also see the discussion on #2699, for a more complete fix.
---
 proxy/http/HttpSM.cc | 10 +---------
 1 file changed, 1 insertion(+), 9 deletions(-)

diff --git a/proxy/http/HttpSM.cc b/proxy/http/HttpSM.cc
index 165a1c1..f2a6f10 100644
--- a/proxy/http/HttpSM.cc
+++ b/proxy/http/HttpSM.cc
@@ -2872,15 +2872,7 @@ bool
 HttpSM::is_http_server_eos_truncation(HttpTunnelProducer *p)
 {
   if ((p->do_dechunking || p->do_chunked_passthru) && p->chunked_handler.truncation) {
-    // TS-3054 - In the chunked cases, chunked data that is incomplete
-    // should not be cached, but it should be passed onto the client
-    // This makes ATS more transparent in the case of non-standard
-    // servers.  The cache aborts are dealt with in other checks
-    // on the truncation flag elsewhere in the code.  This return value
-    // invalidates the current data being passed over to the client.
-    // So changing it from return true to return false, so the partial data
-    // is passed onto the client.
-    return false;
+    return true;
   }
 
   //////////////////////////////////////////////////////////////

-- 
To stop receiving notification emails like this one, please contact
['"commits@trafficserver.apache.org" <co...@trafficserver.apache.org>'].