Fix pre-condition precedence

According to RFC7232 (https://tools.ietf.org/html/rfc7232#section-6), If-None-Match Must be checked before If-Modidied-Since
slimphp · Feb 28, 2019 · e56d164 · e56d164
1 parent 1ebc898
commit e56d164
Showing 1 changed file with 16 additions and 15 deletions.
diff --git a/src/Cache.php b/src/Cache.php
@@ -72,21 +72,6 @@ public function __invoke(RequestInterface $request, ResponseInterface $response,
             }
         }
 
-        // Last-Modified header and conditional GET check
-        $lastModified = $response->getHeaderLine('Last-Modified');
-
-        if ($lastModified) {
-            if (!is_integer($lastModified)) {
-                $lastModified = strtotime($lastModified);
-            }
-
-            $ifModifiedSince = $request->getHeaderLine('If-Modified-Since');
-
-            if ($ifModifiedSince && $lastModified <= strtotime($ifModifiedSince)) {
-                return $response->withStatus(304);
-            }
-        }
-
         // ETag header and conditional GET check
         $etag = $response->getHeader('ETag');
         $etag = reset($etag);
@@ -102,6 +87,22 @@ public function __invoke(RequestInterface $request, ResponseInterface $response,
             }
         }
 
+
+        // Last-Modified header and conditional GET check
+        $lastModified = $response->getHeaderLine('Last-Modified');
+
+        if ($lastModified) {
+            if (!is_integer($lastModified)) {
+                $lastModified = strtotime($lastModified);
+            }
+
+            $ifModifiedSince = $request->getHeaderLine('If-Modified-Since');
+
+            if ($ifModifiedSince && $lastModified <= strtotime($ifModifiedSince)) {
+                return $response->withStatus(304);
+            }
+        }
+
         return $response;
     }
 }