[JENKINS-18870] JENKINS_ENABLE_ACCESS_LOG for Debian

The rpm provide JENKINS_ENABLE_ACCESS_LOG which enable Jenkins to write down some access log. It is missing from the debian package. 1) add JENKINS_ENABLE_ACCESS_LOG (default: "no") to debian/jenkins.default. 1) add JENKINS_ENABLE_ACCESS_LOG support in debian/jenkins.init based on what is done for rpm/suse. 3) update the logrotate script. Signed-off-by: Antoine Musso <hashar@free.fr>
jenkinsci · Nov 20, 2017 · 0913729 · 0913729
1 parent e447477
commit 0913729
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 1 deletion.
diff --git a/deb/build/debian/jenkins.default b/deb/build/debian/jenkins.default
@@ -34,6 +34,10 @@ RUN_STANDALONE=true
 JENKINS_LOG=/var/log/$NAME/$NAME.log
 #JENKINS_LOG=daemon.info
 
+# Whether to enable web access logging or not.
+# Set to "yes" to enable logging to /var/log/$NAME/access_log
+JENKINS_ENABLE_ACCESS_LOG="no"
+
 # OS LIMITS SETUP
 #   comment this out to observe /etc/security/limits.conf
 #   this is on by default because http://github.com/jenkinsci/jenkins/commit/2fb288474e980d0e7ff9c4a3b768874835a3e92e

diff --git a/deb/build/debian/jenkins.init b/deb/build/debian/jenkins.init
@@ -31,6 +31,9 @@ JAVA_ALLOWED_VERSION="18"
 if [ -n "$UMASK" ]; then
     DAEMON_ARGS="$DAEMON_ARGS --umask=$UMASK"
 fi
+if [ "$JENKINS_ENABLE_ACCESS_LOG" = "yes" ]; then
+    DAEMON_ARGS="$DAEMON_ARGS --accessLoggerClassName=winstone.accesslog.SimpleAccessLogger --simpleAccessLogger.format=combined --simpleAccessLogger.file=/var/log/$NAME/access_log"
+fi
 
 SU=/bin/su
 

diff --git a/deb/build/debian/jenkins.logrotate b/deb/build/debian/jenkins.logrotate
@@ -1,4 +1,4 @@
-/var/log/@@ARTIFACTNAME@@/@@ARTIFACTNAME@@.log {
+/var/log/@@ARTIFACTNAME@@/@@ARTIFACTNAME@@.log /var/log/@@ARTIFACTNAME@@/access_log {
         weekly
         copytruncate
         missingok