Remove ie_refresh configuration option

This option was provided as a hack to workaround problems in MSIE 5.01
and older.

Since those MSIE versions are long deprecated and no longer even
registering on the popularity charts for more than 5 years I think its
time we removed this hack.

diff --git a/doc/release-notes/release-4.sgml b/doc/release-notes/release-4.sgml
index f4ad326697309d77781a55184e5aa61a1bbee65c..2a92633ec60779115724c6da44835b60bb18894a 100644 (file)
--- a/doc/release-notes/release-4.sgml
+++ b/doc/release-notes/release-4.sgml
@@ -343,6 +343,9 @@ This section gives a thorough account of those changes in three categories:
        <p>Superceded by <em>cache_peer_access</em>. Use dstdomain ACL
           in the access control list to restrict domains requested.
 
+       <tag>ie_refresh</tag>
+       <p>Removed. MSIE 3.x, 4.x, 5.0 and 5.01 are no longer popular browsers.
+
        <tag>sslproxy_cafile</tag>
        <p>Replaced by <em>tls_outgoing_options cafile=</em>.
           Which now takes multiple entries.

diff --git a/src/SquidConfig.h b/src/SquidConfig.h
index 648054159ec1fedc09dd1a7e1f3a086057500203..3ee1247898635fee35804dda90ac466d4aaefbe3 100644 (file)
--- a/src/SquidConfig.h
+++ b/src/SquidConfig.h
@@ -305,7 +305,6 @@ public:
         int digest_generation;
 #endif
 
-        int ie_refresh;
         int vary_ignore_expire;
         int surrogate_is_remote;
         int request_entities;

diff --git a/src/cf.data.pre b/src/cf.data.pre
index 65ad78bc5aff303a31bdfc4583e244c3a36fc2a3..26fe3786b0733d13518b15e6533d86d007b6516d 100644 (file)
--- a/src/cf.data.pre
+++ b/src/cf.data.pre
@@ -163,13 +163,19 @@ DOC_START
        This option is not yet supported by Squid-3.
 DOC_END
 
-# Options removed in 3.6
+# Options removed in 4.x
 NAME: cache_peer_domain cache_host_domain
 TYPE: obsolete
 DOC_START
        Replace with dstdomain ACLs and cache_peer_access.
 DOC_END
 
+NAME: ie_refresh
+TYPE: obsolete
+DOC_START
+       Remove this line. The behaviour enabled by this is no longer needed.
+DOC_END
+
 NAME: sslproxy_cafile
 TYPE: obsolete
 DOC_START
@@ -5889,30 +5895,6 @@ DOC_START
        replies as required by RFC2616.
 DOC_END
 
-NAME: ie_refresh
-COMMENT: on|off
-TYPE: onoff
-LOC: Config.onoff.ie_refresh
-DEFAULT: off
-DOC_START
-       Microsoft Internet Explorer up until version 5.5 Service
-       Pack 1 has an issue with transparent proxies, wherein it
-       is impossible to force a refresh.  Turning this on provides
-       a partial fix to the problem, by causing all IMS-REFRESH
-       requests from older IE versions to check the origin server
-       for fresh content.  This reduces hit ratio by some amount
-       (~10% in my experience), but allows users to actually get
-       fresh content when they want it.  Note because Squid
-       cannot tell if the user is using 5.5 or 5.5SP1, the behavior
-       of 5.5 is unchanged from old versions of Squid (i.e. a
-       forced refresh is impossible).  Newer versions of IE will,
-       hopefully, continue to have the new behavior and will be
-       handled based on that assumption.  This option defaults to
-       the old Squid behavior, which is better for hit ratios but
-       worse for clients using IE, if they need to be able to
-       force fresh content.
-DOC_END
-
 NAME: vary_ignore_expire
 COMMENT: on|off
 TYPE: onoff

diff --git a/src/client_side_request.cc b/src/client_side_request.cc
index d8da5694124e3dc10e9b2e96d7347a17514d4e90..c7b307e340a9a772459699568de8327b88d00f4c 100644 (file)
--- a/src/client_side_request.cc
+++ b/src/client_side_request.cc
@@ -1072,28 +1072,6 @@ clientInterpretRequestHeaders(ClientHttpRequest * http)
             // RFC 2616: treat Pragma:no-cache as if it was Cache-Control:no-cache when Cache-Control is missing
         } else if (req_hdr->has(Http::HdrType::PRAGMA))
             no_cache = req_hdr->hasListMember(Http::HdrType::PRAGMA,"no-cache",',');
-
-        /*
-        * Work around for supporting the Reload button in IE browsers when Squid
-        * is used as an accelerator or transparent proxy, by turning accelerated
-        * IMS request to no-cache requests. Now knows about IE 5.5 fix (is
-        * actually only fixed in SP1, but we can't tell whether we are talking to
-        * SP1 or not so all 5.5 versions are treated 'normally').
-        */
-        if (Config.onoff.ie_refresh) {
-            if (http->flags.accel && request->flags.ims) {
-                if ((str = req_hdr->getStr(Http::HdrType::USER_AGENT))) {
-                    if (strstr(str, "MSIE 5.01") != NULL)
-                        no_cache=true;
-                    else if (strstr(str, "MSIE 5.0") != NULL)
-                        no_cache=true;
-                    else if (strstr(str, "MSIE 4.") != NULL)
-                        no_cache=true;
-                    else if (strstr(str, "MSIE 3.") != NULL)
-                        no_cache=true;
-                }
-            }
-        }
     }
 
     if (request->method == Http::METHOD_OTHER) {