You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by zw...@apache.org on 2013/09/10 17:10:05 UTC

[39/48] git commit: TS-2176: do not reset value of api_skip_cache_lookup when reading it

TS-2176: do not reset value of api_skip_cache_lookup when reading it

When is_request_cache_lookupable reads the value of api_skip_cache_lookup
it also resets the value to false.  Since the function is called
from HandleRequest, the flag set by the API call will be ignored
by DecideCacheLookup.


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

Branch: refs/heads/5.0.x
Commit: b1845c4be987a2d0ba331b46839c01e0f17d7666
Parents: f1bffdf
Author: Corey Cossentino <co...@omniti.com>
Authored: Wed Sep 4 13:04:05 2013 -0400
Committer: James Peach <jp...@apache.org>
Committed: Mon Sep 9 15:28:57 2013 -0700

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


http://git-wip-us.apache.org/repos/asf/trafficserver/blob/b1845c4b/CHANGES
----------------------------------------------------------------------
diff --git a/CHANGES b/CHANGES
index f63e647..9027adb 100644
--- a/CHANGES
+++ b/CHANGES
@@ -1,6 +1,10 @@
                                                          -*- coding: utf-8 -*-
 Changes with Apache Traffic Server 4.1.0
-  
+
+
+  *) [TS-2176] do not reset value of api_skip_cache_lookup when reading it
+    Author: Corey Cossentino <co...@omniti.com>
+
   *) [TS-2191] Do not reschedule http_sm when the sm_list`s lock is not
    acquired. This can lose items for the WebUI, which will be fixed in a
    separate bug.

http://git-wip-us.apache.org/repos/asf/trafficserver/blob/b1845c4b/proxy/http/HttpTransact.cc
----------------------------------------------------------------------
diff --git a/proxy/http/HttpTransact.cc b/proxy/http/HttpTransact.cc
index 512a1cc..a0c9176 100644
--- a/proxy/http/HttpTransact.cc
+++ b/proxy/http/HttpTransact.cc
@@ -5886,7 +5886,6 @@ HttpTransact::is_request_cache_lookupable(State* s)
   }
 
   if (s->api_skip_cache_lookup) {
-    s->api_skip_cache_lookup = false;
     return false;
   }
   // Even with "no-cache" directive, we want to do a cache lookup