You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by co...@hyperreal.org on 1999/05/03 17:09:08 UTC
cvs commit: apache-1.3/src/main http_protocol.c
coar 99/05/03 08:09:08
Modified: src CHANGES
src/main http_protocol.c
Log:
If we're removing Vary from the response header, force the response
to HTTP/1.0 for cache friendliness.
Submitted by: Dmitry Khrustalev <di...@zippy.machaon.ru>
Reviewed by: Ken Coar
Revision Changes Path
1.1341 +1 -1 apache-1.3/src/CHANGES
Index: CHANGES
===================================================================
RCS file: /home/cvs/apache-1.3/src/CHANGES,v
retrieving revision 1.1340
retrieving revision 1.1341
diff -u -r1.1340 -r1.1341
--- CHANGES 1999/05/03 10:23:44 1.1340
+++ CHANGES 1999/05/03 15:09:04 1.1341
@@ -34,7 +34,7 @@
*) Add "force-no-vary" envariable to allow servers to work around
clients that choke on "Vary" fields in the response header.
- [Ken Coar] PR#4118
+ [Ken Coar, Dmitry Khrustalev <di...@zippy.machaon.ru>] PR#4118
*) Fixed a bug in mod_dir that causes a child process will infinitely
recurse when it attemps to handle a request for a directory wnd the
1.267 +4 -0 apache-1.3/src/main/http_protocol.c
Index: http_protocol.c
===================================================================
RCS file: /home/cvs/apache-1.3/src/main/http_protocol.c,v
retrieving revision 1.266
retrieving revision 1.267
diff -u -r1.266 -r1.267
--- http_protocol.c 1999/04/29 15:06:52 1.266
+++ http_protocol.c 1999/05/03 15:09:07 1.267
@@ -1476,9 +1476,13 @@
/*
* Remove the 'Vary' header field if the client can't handle it.
+ * Since this will have nasty effects on HTTP/1.1 caches, force
+ * the response into HTTP/1.0 mode.
*/
if (ap_table_get(r->subprocess_env, "force-no-vary") != NULL) {
ap_table_unset(r->headers_out, "Vary");
+ r->proto_num = HTTP_VERSION(1,0);
+ ap_table_set(r->subprocess_env, "force-response-1.0", "1");
}
ap_hard_timeout("send headers", r);