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 2011/06/07 18:09:52 UTC
svn commit: r1133066 -
/trafficserver/traffic/trunk/proxy/http/HttpTransact.cc
Author: zwoop
Date: Tue Jun 7 16:09:52 2011
New Revision: 1133066
URL: http://svn.apache.org/viewvc?rev=1133066&view=rev
Log:
TS-825 negative caching caches responses that should never be cached
Author: William Bardwell
Review: leif
Modified:
trafficserver/traffic/trunk/proxy/http/HttpTransact.cc
Modified: trafficserver/traffic/trunk/proxy/http/HttpTransact.cc
URL: http://svn.apache.org/viewvc/trafficserver/traffic/trunk/proxy/http/HttpTransact.cc?rev=1133066&r1=1133065&r2=1133066&view=diff
==============================================================================
--- trafficserver/traffic/trunk/proxy/http/HttpTransact.cc (original)
+++ trafficserver/traffic/trunk/proxy/http/HttpTransact.cc Tue Jun 7 16:09:52 2011
@@ -6324,6 +6324,10 @@ HttpTransact::is_response_cacheable(Stat
return false;
}
}
+ if (response_code == HTTP_STATUS_SEE_OTHER ||
+ response_code == HTTP_STATUS_UNAUTHORIZED ||
+ response_code == HTTP_STATUS_PROXY_AUTHENTICATION_REQUIRED)
+ return false;
// let is_negative_caching_approriate decide what to do
return true;
/* Since we weren't caching response obtained with