From: Galen Charlton <gmc@esilibrary.com>
Date: Wed, 17 Sep 2014 17:22:15 +0000 (-0700)
Subject: LP#1370630: don't log mod_deflate actions by default
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=refs%2Fheads%2Fuser%2Fgmcharlt%2Flp1370630_remove_deflate_log;p=working%2FEvergreen.git

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>
---

diff --git a/Open-ILS/examples/apache/eg_vhost.conf.in b/Open-ILS/examples/apache/eg_vhost.conf.in
index 2f077ea0b2..6f3fae3ccc 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 a44b9ada09..53275ae9a6 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