Skip to content

Commit

Permalink
Ensure the Docker nginx user can write servers.json. Fixes #1046
Browse files Browse the repository at this point in the history
  • Loading branch information
Jinna Kiisuo committed Jan 29, 2024
1 parent b12bb6c commit 9854c45
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,13 @@ ARG UID=101
LABEL maintainer="Alejandro Celaya <[email protected]>"
USER root
RUN rm -r /usr/share/nginx/html && rm /etc/nginx/conf.d/default.conf
USER $UID
COPY config/docker/nginx.conf /etc/nginx/conf.d/default.conf
COPY scripts/docker/servers_from_env.sh /docker-entrypoint.d/30-shlink-servers-json.sh
COPY --from=node /shlink-web-client/build /usr/share/nginx/html

# Allow the nginx user to create new files in the html dir, but not modify existing files.
# This is required by 30-shlink-servers-json.sh
RUN chown :$UID /usr/share/nginx/html && chmod g+wx /usr/share/nginx/html

# Switch to non-privileged UID as the last step
USER $UID

0 comments on commit 9854c45

Please sign in to comment.