From 80b9bc27ecd3be475562567fa73d72c2f451a184 Mon Sep 17 00:00:00 2001
From: Galen Charlton <gmc@esilibrary.com>
Date: Wed, 17 Sep 2014 10:22:15 -0700
Subject: [PATCH] LP#1370630: don't log mod_deflate actions by default

This patch comments out logging of mod_deflate actions,
as this is needed only for debugging.  Otherwise,
deflate_log will grow without bound, as it isn't dealt
with by typical default logrotate configs.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Ben Shum <bshum@biblio.org>
---
 Open-ILS/examples/apache/eg_vhost.conf.in    | 15 ++++++++-------
 Open-ILS/examples/apache_24/eg_vhost.conf.in | 14 ++++++++------
 2 files changed, 16 insertions(+), 13 deletions(-)

diff --git a/Open-ILS/examples/apache/eg_vhost.conf.in b/Open-ILS/examples/apache/eg_vhost.conf.in
index d1b48f0b90..267331e2f8 100644
--- a/Open-ILS/examples/apache/eg_vhost.conf.in
+++ b/Open-ILS/examples/apache/eg_vhost.conf.in
@@ -743,15 +743,16 @@ RewriteRule ^/openurl$ ${openurl:%1} [NE,PT]
 </LocationMatch>
 
 # ----------------------------------------------------------------------------------
-# Some mod_deflate logging setup
+# Some mod_deflate setup
 # ----------------------------------------------------------------------------------
 <IfModule mod_deflate.c>
-    DeflateFilterNote Input instream
-    DeflateFilterNote Output outstream
-    DeflateFilterNote Ratio ratio
-
-    LogFormat '"%r" %{outstream}n/%{instream}n (%{ratio}n%%)' deflate
-    CustomLog /var/log/apache2/deflate_log deflate
+    ## optional logging for mod_deflate debugging
+    ##DeflateFilterNote Input instream
+    ##DeflateFilterNote Output outstream
+    ##DeflateFilterNote Ratio ratio
+    ##
+    ##LogFormat '"%r" %{outstream}n/%{instream}n (%{ratio}n%%)' deflate
+    ##CustomLog /var/log/apache2/deflate_log deflate
 
     # There are problems with XMLENT and mod_deflate - so lets disable it
     # This is where we don't have a pre-existing LocationMatch directive earlier
diff --git a/Open-ILS/examples/apache_24/eg_vhost.conf.in b/Open-ILS/examples/apache_24/eg_vhost.conf.in
index f6e65e7abe..a21df11875 100644
--- a/Open-ILS/examples/apache_24/eg_vhost.conf.in
+++ b/Open-ILS/examples/apache_24/eg_vhost.conf.in
@@ -755,15 +755,17 @@ RewriteRule ^/openurl$ ${openurl:%1} [NE,PT]
 </LocationMatch>
 
 # ----------------------------------------------------------------------------------
-# Some mod_deflate logging setup
+# Some mod_deflate setup
 # ----------------------------------------------------------------------------------
 <IfModule mod_deflate.c>
-    DeflateFilterNote Input instream
-    DeflateFilterNote Output outstream
-    DeflateFilterNote Ratio ratio
 
-    LogFormat '"%r" %{outstream}n/%{instream}n (%{ratio}n%%)' deflate
-    CustomLog /var/log/apache2/deflate_log deflate
+    ## optional logging for mod_deflate debugging
+    ##DeflateFilterNote Input instream
+    ##DeflateFilterNote Output outstream
+    ##DeflateFilterNote Ratio ratio
+    ##
+    ##LogFormat '"%r" %{outstream}n/%{instream}n (%{ratio}n%%)' deflate
+    ##CustomLog /var/log/apache2/deflate_log deflate
 
     # There are problems with XMLENT and mod_deflate - so lets disable it
     # This is where we don't have a pre-existing LocationMatch directive earlier
-- 
2.11.0