You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tapestry.apache.org by hl...@apache.org on 2011/10/24 18:15:52 UTC

svn commit: r1188195 - /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/assets/ResourceChangeTrackerImpl.java

Author: hlship
Date: Mon Oct 24 16:15:52 2011
New Revision: 1188195

URL: http://svn.apache.org/viewvc?rev=1188195&view=rev
Log:
TAP5-1717: Assets get cache expiry header of year 1979 in production mode

Modified:
    tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/assets/ResourceChangeTrackerImpl.java

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/assets/ResourceChangeTrackerImpl.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/assets/ResourceChangeTrackerImpl.java?rev=1188195&r1=1188194&r2=1188195&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/assets/ResourceChangeTrackerImpl.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/assets/ResourceChangeTrackerImpl.java Mon Oct 24 16:15:52 2011
@@ -29,6 +29,13 @@ public class ResourceChangeTrackerImpl e
 {
     private final URLChangeTracker tracker;
 
+    /**
+     * Used in production mode as the last modified time of any resource exposed to the client. Remember that
+     * all exposed assets include a URL with a version number, and each new deployment of the application should change
+     * that version number.
+     */
+    private final long fixedLastModifiedTime = System.currentTimeMillis();
+
     public ResourceChangeTrackerImpl(ClasspathURLConverter classpathURLConverter,
                                      @Symbol(SymbolConstants.PRODUCTION_MODE)
                                      boolean productionMode)
@@ -51,7 +58,7 @@ public class ResourceChangeTrackerImpl e
     {
         if (tracker == null)
         {
-            return 0;
+            return fixedLastModifiedTime;
         }
 
         return tracker.add(resource.toURL());