From 18880befdb1c96b07439dfe26422c1427f9edb12 Mon Sep 17 00:00:00 2001
From: Robert Wlodarczyk <robert@simplicityguy.com>
Date: Sun, 5 Jan 2025 10:52:44 -0800
Subject: [PATCH] fix: LOG_DIR is not used anywhere, remove it.

---
 Dockerfile    | 4 +---
 entrypoint.sh | 8 --------
 2 files changed, 1 insertion(+), 11 deletions(-)

diff --git a/Dockerfile b/Dockerfile
index 16fe7f6..b6becaa 100644
--- a/Dockerfile
+++ b/Dockerfile
@@ -48,10 +48,8 @@ RUN apk update --quiet && \
         wget && \
     rm /var/cache/apk/* && \
     mkdir -p ${HOME_DIR}/jobs && \
-    mkdir -p /var/log/crontab && \
     rm -rf /etc/periodic /etc/crontabs/root && \
-    adduser -S docker -D && \
-    chmod u=rwx,go=rw /var/log/crontab
+    adduser -S docker -D
 
 USER docker
 
diff --git a/entrypoint.sh b/entrypoint.sh
index 3df3fde..bc3892f 100755
--- a/entrypoint.sh
+++ b/entrypoint.sh
@@ -4,12 +4,10 @@ set -e
 
 DOCKER_SOCK=/var/run/docker.sock
 CRONTAB_FILE=/etc/crontabs/docker
-LOG_DIR=/var/log/crontab
 
 if [ -z "${HOME_DIR}" ] && [ -n "${TEST_MODE}" ]; then
     HOME_DIR=/tmp/crontab-docker-testing
     CRONTAB_FILE=${HOME_DIR}/test
-    LOG_DIR=${HOME_DIR}/crontab
 elif [ -z "${HOME_DIR}" ]; then
     echo "HOME_DIR not set."
     exit 1
@@ -22,12 +20,6 @@ if [ -z "${DOCKER_HOST}" ] && [ -a "${DOCKER_PORT_2375_TCP}" ]; then
     export DOCKER_HOST="tcp://docker:2375"
 fi
 
-if [ "${LOG_FILE}" == "" ]; then
-    LOG_FILE=${LOG_DIR}/jobs.log
-    mkdir -p "${LOG_DIR}"
-    touch "${LOG_FILE}"
-fi
-
 normalize_config() {
     JSON_CONFIG={}
     if [ -f "${HOME_DIR}/config.json" ]; then