[Docker] Split config dir creation/set permissions

If the config dir already exists, setting the mode (`-m 777`) doesn't
actually work as it should. This change splits the command into two
separate commands for directory creation and enabling the directory to
be writable by all.

Fixes #658
main
Ben Busby 2022-02-21 09:33:30 -07:00
parent 9984158ec1
commit e72d8437f7
No known key found for this signature in database
GPG Key ID: B9B7231E01D924A1
1 changed files with 3 additions and 2 deletions

View File

@ -22,7 +22,8 @@ RUN apk -U upgrade
ARG DOCKER_USER=whoogle ARG DOCKER_USER=whoogle
ARG DOCKER_USERID=927 ARG DOCKER_USERID=927
ARG config_dir=/config ARG config_dir=/config
RUN mkdir -p -m 777 $config_dir RUN mkdir -p $config_dir
RUN chmod a+w $config_dir
VOLUME $config_dir VOLUME $config_dir
ARG username='' ARG username=''
@ -75,7 +76,7 @@ COPY run .
RUN adduser -D -g $DOCKER_USERID -u $DOCKER_USERID $DOCKER_USER RUN adduser -D -g $DOCKER_USERID -u $DOCKER_USERID $DOCKER_USER
# Fix ownership / permissions # Fix ownership / permissions
RUN chown -R ${DOCKER_USER}:${DOCKER_USER} /whoogle /var/lib/tor RUN chown -R ${DOCKER_USER}:${DOCKER_USER} /whoogle /var/lib/tor
# Allow writing symlinks to build dir # Allow writing symlinks to build dir
RUN chown $DOCKER_USERID:$DOCKER_USERID app/static/build RUN chown $DOCKER_USERID:$DOCKER_USERID app/static/build