Do not update stored headers on HTTP 304 responses (#267)

... until we learn how to synchronize stored and in-transit metadata.

This temporary disables a feature first introduced in commit abf396e.

diff --git a/src/store/Controller.cc b/src/store/Controller.cc
index 3ed7b5ad402937a0127216a220a7356b2d10d0d8..68d2b4a869e1d6548e530f2a0d8699831fd6e2ca 100644 (file)
--- a/src/store/Controller.cc
+++ b/src/store/Controller.cc
@@ -688,13 +688,9 @@ Store::Controller::updateOnNotModified(StoreEntry *old, const StoreEntry &newer)
     if (!old->timestampsSet() && !modified)
         return;
 
-    /* update stored image of the old entry */
-
-    if (sharedMemStore && old->mem_status == IN_MEMORY && !EBIT_TEST(old->flags, ENTRY_SPECIAL))
-        sharedMemStore->updateHeaders(old);
-
-    if (old->hasDisk())
-        swapDir->updateHeaders(old);
+    // XXX: Call memStore->updateHeaders(old) and swapDir->updateHeaders(old) to
+    // update stored headers, stored metadata, and in-transit metadata.
+    debugs(20, 3, *old << " headers were modified: " << modified);
 }
 
 bool