You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by wa...@apache.org on 2008/10/13 15:23:11 UTC

svn commit: r704116 - /felix/trunk/http.jetty/src/main/java/org/mortbay/jetty/servlet/OsgiResourceHolder.java

Author: walkerr
Date: Mon Oct 13 06:23:10 2008
New Revision: 704116

URL: http://svn.apache.org/viewvc?rev=704116&view=rev
Log:
FELIX-407 - minor fix to ensure resource is returned in a case where modified timestamp of resource can't be determined

Modified:
    felix/trunk/http.jetty/src/main/java/org/mortbay/jetty/servlet/OsgiResourceHolder.java

Modified: felix/trunk/http.jetty/src/main/java/org/mortbay/jetty/servlet/OsgiResourceHolder.java
URL: http://svn.apache.org/viewvc/felix/trunk/http.jetty/src/main/java/org/mortbay/jetty/servlet/OsgiResourceHolder.java?rev=704116&r1=704115&r2=704116&view=diff
==============================================================================
--- felix/trunk/http.jetty/src/main/java/org/mortbay/jetty/servlet/OsgiResourceHolder.java (original)
+++ felix/trunk/http.jetty/src/main/java/org/mortbay/jetty/servlet/OsgiResourceHolder.java Mon Oct 13 06:23:10 2008
@@ -306,8 +306,9 @@
         modSince /= 1000;
         resTimestamp /= 1000;
         
-        // Timestamp check to see if modified
-        if (modSince == -1 || resTimestamp > modSince)
+        // Timestamp check to see if modified - resTimestamp 0 check is for 
+        // safety in case we didn't manage to get a timestamp for the resource
+        if (resTimestamp == 0 || modSince == -1 || resTimestamp > modSince)
         {
             retval = true;
         }