From 2d601faaa0df02a64fd5378e52dc4c61b6665458 Mon Sep 17 00:00:00 2001
From: nezu <29180158+dumbasPL@users.noreply.github.com>
Date: Mon, 4 Nov 2024 19:20:45 +0100
Subject: [PATCH] fix: remove hardcoded HOSTNAME in Docker

Instead of hardcoding it in supervisord.conf, we simply set it to an
empty value in the Dockerfile. This prevents docker from setting it's
own value while also allowing the user to override it.

In next JS an empty HOSTNAME is equivalent to not setting it at all.
---
 Dockerfile              | 1 +
 docker/supervisord.conf | 1 -
 2 files changed, 1 insertion(+), 1 deletion(-)

diff --git a/Dockerfile b/Dockerfile
index 62386b65..c59bf812 100644
--- a/Dockerfile
+++ b/Dockerfile
@@ -31,6 +31,7 @@ ARG UID
 ARG GID
 
 ENV NODE_ENV production
+ENV HOSTNAME=
 
 RUN echo 'deb http://deb.debian.org/debian bookworm-backports main' >> /etc/apt/sources.list
 RUN apt-get update && apt-get install -y \
diff --git a/docker/supervisord.conf b/docker/supervisord.conf
index 59a02aa0..0930b956 100644
--- a/docker/supervisord.conf
+++ b/docker/supervisord.conf
@@ -14,7 +14,6 @@ stdout_logfile_backups=5
 redirect_stderr=true
 
 [program:borgwarehouse]
-environment=HOSTNAME=0.0.0.0
 command=/usr/local/bin/node server.js
 stdout_logfile=/home/borgwarehouse/tmp/borgwarehouse.log
 stdout_logfile_maxbytes=10MB