Updated Branches:
  refs/heads/master ed8f6e9b0 -> 46adeb3d0

TS-1468: Check vary and accept headers on non-200 responses in cache


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

Branch: refs/heads/master
Commit: 46adeb3d0e4b8a927c24a479f2c1a91b28358c6e
Parents: ed8f6e9
Author: Bryan Call <[email protected]>
Authored: Wed Nov 20 18:49:00 2013 -0800
Committer: Bryan Call <[email protected]>
Committed: Wed Nov 20 18:49:00 2013 -0800

----------------------------------------------------------------------
 CHANGES                         | 2 ++
 proxy/http/HttpTransactCache.cc | 5 -----
 2 files changed, 2 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/trafficserver/blob/46adeb3d/CHANGES
----------------------------------------------------------------------
diff --git a/CHANGES b/CHANGES
index d2fa182..fdac461 100644
--- a/CHANGES
+++ b/CHANGES
@@ -1,6 +1,8 @@
                                                          -*- coding: utf-8 -*-
 Changes with Apache Traffic Server 4.2.0
 
+  *) [TS-1468] Check vary and accept headers on non-200 responses in cache
+
   *) [TS-2352] refine THREAD_ALLOC feature.
 
   *) [TS-2364] Introduce slice notation to field syntax in log format.

http://git-wip-us.apache.org/repos/asf/trafficserver/blob/46adeb3d/proxy/http/HttpTransactCache.cc
----------------------------------------------------------------------
diff --git a/proxy/http/HttpTransactCache.cc b/proxy/http/HttpTransactCache.cc
index 8699ee8..1b0fc04 100644
--- a/proxy/http/HttpTransactCache.cc
+++ b/proxy/http/HttpTransactCache.cc
@@ -305,11 +305,6 @@ 
HttpTransactCache::calculate_quality_of_match(CacheLookupHttpConfig * http_confi
   if (client_request->method_get_wksidx() == HTTP_WKSIDX_PURGE)
     return (float)1.0;
 
-  // BZ49848 - for cached negative respones, we don't check for the
-  // Accept* headers. This should also be good for the 301 response.
-  if (obj_origin_server_response->status_get() != HTTP_STATUS_OK)
-    return (float)1.0;
-
   // Now calculate a quality based on all sorts of logic
   float q[4], Q;
   MIMEField *accept_field;

Reply via email to