Re: mod_deflate patch

[William A. Rowe, Jr.]

This patch is correct.  +1 to any/all branches as applicable.

Brian J. France wrote:

This was talked about a few weeks back but I don't think anything ever came of it. The patch below would allow mod_deflate to compress internal redirects while still skipping sub requests.

I have been running this on my personal server for a few weeks now with no issues.

Let the voting begin.

Brian

Index: modules/filters/mod_deflate.c
===================================================================
--- modules/filters/mod_deflate.c       (revision 394996)
+++ modules/filters/mod_deflate.c       (working copy)
@@ -240,7 +240,7 @@
         const char *encoding;
         /* only work on main request/no subrequests */
-        if (!ap_is_initial_req(r)) {
+        if (r->main != NULL) {
             ap_remove_output_filter(f);
             return ap_pass_brigade(f->next, bb);
         }