From d19e92ea550b9ac9c4568a953ed60aac7e8ff3bc Mon Sep 17 00:00:00 2001
From: Joseph Hale <me@jhale.dev>
Date: Tue, 17 Feb 2026 14:38:53 -0700
Subject: [PATCH] Use /home/joseph variable instead of hardcoded /tmp paths

Move ENV HOME=/tmp to earlier in Dockerfile so it can be used
in subsequent RUN commands. Replace hardcoded /tmp paths with
/home/joseph variable for consistency and flexibility.
---
 Dockerfile | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/Dockerfile b/Dockerfile
index a5e4210..04c86d6 100644
--- a/Dockerfile
+++ b/Dockerfile
@@ -18,13 +18,16 @@ RUN ARCH=$(uname -m) && \
     fi && \
     chmod +x /usr/local/bin/speedtest
 
+# Set HOME environment variable for use in subsequent commands
+ENV HOME=/tmp
+
 # Create directories with wide permissions for any UID
-RUN mkdir -p /data /app /tmp/.config/ookla && \
-    chmod 777 /data /tmp && \
-    chmod -R 777 /tmp/.config
+RUN mkdir -p /data /app $HOME/.config/ookla && \
+    chmod 777 /data $HOME && \
+    chmod -R 777 $HOME/.config
 
 # Accept license automatically (will be copied to user's home at runtime)
-RUN echo '{"Settings": {"LicenseAccepted": "604ec14274326065ea260afd5035643b"}}' > /tmp/speedtest-cli.json
+RUN echo '{"Settings": {"LicenseAccepted": "604ec14274326065ea260afd5035643b"}}' > $HOME/speedtest-cli.json
 
 # Copy application files
 COPY speedtest_runner.sh /app/
@@ -42,7 +45,6 @@ RUN sed -i 's/listen       80;/listen       8080;/g' /etc/nginx/conf.d/default.c
 ENV CRON_SCHEDULE="*/30 * * * *"
 ENV DATA_FILE="/data/speedtest.csv"
 ENV HTML_FILE="/usr/share/nginx/html/index.html"
-ENV HOME=/tmp
 
 # Expose web port
 EXPOSE 8080
-- 
2.51.2