You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@shindig.apache.org by zh...@apache.org on 2010/10/05 01:01:48 UTC
svn commit: r1004475 -
/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/http/AbstractHttpCache.java
Author: zhoresh
Date: Mon Oct 4 23:01:48 2010
New Revision: 1004475
URL: http://svn.apache.org/viewvc?rev=1004475&view=rev
Log:
Remove final from getResponse to allow override
Modified:
shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/http/AbstractHttpCache.java
Modified: shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/http/AbstractHttpCache.java
URL: http://svn.apache.org/viewvc/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/http/AbstractHttpCache.java?rev=1004475&r1=1004474&r2=1004475&view=diff
==============================================================================
--- shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/http/AbstractHttpCache.java (original)
+++ shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/http/AbstractHttpCache.java Mon Oct 4 23:01:48 2010
@@ -42,7 +42,7 @@ public abstract class AbstractHttpCache
protected abstract void addResponseImpl(String key, HttpResponse response);
protected abstract HttpResponse removeResponseImpl(String key);
- public final HttpResponse getResponse(HttpRequest request) {
+ public HttpResponse getResponse(HttpRequest request) {
if (isCacheable(request)) {
String keyString = createKey(request);
HttpResponse cached = getResponseImpl(keyString);
@@ -98,7 +98,7 @@ public abstract class AbstractHttpCache
// Caching was forced. Ignore what the response wants.
return true;
}
-
+
if (response.getHttpStatusCode() == HttpResponse.SC_NOT_MODIFIED) {
// Shindig server will serve 304s. Do not cache 304s from the origin server.
return false;