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=a436abf37a0ac3a8548d0f5f5820b8c04b9f8816;p=Evergreen.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>
Signed-off-by: Ben Shum <bshum@biblio.org>
---

diff --git a/Open-ILS/examples/apache/eg_vhost.conf.in b/Open-ILS/examples/apache/eg_vhost.conf.in
index a6476aae62..07f3295aba 100644
--- a/Open-ILS/examples/apache/eg_vhost.conf.in
+++ b/Open-ILS/examples/apache/eg_vhost.conf.in
@@ -742,15 +742,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 d561916c23..d754568ee5 100644
--- a/Open-ILS/examples/apache_24/eg_vhost.conf.in
+++ b/Open-ILS/examples/apache_24/eg_vhost.conf.in
@@ -754,15 +754,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