From dd8ab9be2950318d90aecccd5934ff75f8451649 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Sandro=20J=C3=A4ckel?= <sandro.jaeckel@gmail.com>
Date: Wed, 3 Jun 2020 07:12:14 +0200
Subject: [PATCH] Fix default uid/gid of node_modules

---
 backend/Dockerfile | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/backend/Dockerfile b/backend/Dockerfile
index d4ce0c5..6d337a4 100644
--- a/backend/Dockerfile
+++ b/backend/Dockerfile
@@ -1,9 +1,9 @@
 FROM alpine:3.12
 
-ENV UID=1000 GID=1000
-RUN export user=youtube \
-  && addgroup -S $user -g $GID && adduser -D -S $user -G $user -u $UID
-USER $user
+ENV UID=1000 \
+  GID=1000 \
+  USER=youtube
+RUN addgroup -S $USER -g $GID && adduser -D -S $USER -G $USER -u $UID
 
 RUN apk add --no-cache \
     ffmpeg \
@@ -13,6 +13,8 @@ RUN apk add --no-cache \
   && apk add --no-cache --repository http://dl-cdn.alpinelinux.org/alpine/edge/testing/ \
     atomicparsley
 
+# switch to default user to not chown node_modules on startup
+USER $USER
 WORKDIR /app
 
 COPY --chown=$UID:$GID [ "package.json", "package-lock.json", "/app/" ]
@@ -21,5 +23,7 @@ RUN npm install
 COPY --chown=$UID:$GID [ "./", "/app/" ]
 
 EXPOSE 17442
+# switch back to root to allow UID/GID changing
+USER root
 ENTRYPOINT [ "/app/entrypoint.sh" ]
 CMD [ "node", "app.js" ]