Author: Philippe Lantin <plantin@cobaltgroup.com>

Bug 2624: Invalid response for IMS request

Squid forgot to verify the client provided If-Modified-Since when
seeing a 304 from upstream.

diff --git a/src/client_side_reply.cc b/src/client_side_reply.cc
index 541abc7ed206df227edbbc476fcda8002220e364..f10ee157575b4f875d8cdab952b7ca0498e3c38a 100644 (file)
--- a/src/client_side_reply.cc
+++ b/src/client_side_reply.cc
@@ -366,7 +366,7 @@ clientReplyContext::handleIMSReply(StoreIOBuffer result)
 
        // if client sent IMS
 
-       if (http->request->flags.ims) {
+        if (http->request->flags.ims && !old_entry->modifiedSince(http->request)) {
            // forward the 304 from origin
            debugs(88, 3, "handleIMSReply: origin replied 304, revalidating existing entry and forwarding 304 to client");
            sendClientUpstreamResponse();