Skip to content

Commit 4fb1b84

Browse files
dumbasPLRavinou
authored andcommitted
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.
1 parent 682ac5f commit 4fb1b84

File tree

2 files changed

+1
-1
lines changed

2 files changed

+1
-1
lines changed

Dockerfile

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@ ARG UID
3131
ARG GID
3232

3333
ENV NODE_ENV production
34+
ENV HOSTNAME=
3435

3536
RUN echo 'deb http://deb.debian.org/debian bookworm-backports main' >> /etc/apt/sources.list
3637
RUN apt-get update && apt-get install -y \

docker/supervisord.conf

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,6 @@ stdout_logfile_backups=5
1414
redirect_stderr=true
1515

1616
[program:borgwarehouse]
17-
environment=HOSTNAME=0.0.0.0
1817
command=/usr/local/bin/node server.js
1918
stdout_logfile=/home/borgwarehouse/tmp/borgwarehouse.log
2019
stdout_logfile_maxbytes=10MB

0 commit comments

Comments
 (0)