Updates php.dockerfile with permissions changes

aschmelyun · web-flow · commit 027616a67c87 · 2021-08-08T01:00:21.000-04:00
Adds lines to pull in docker-compose arguments to env values. Adds a dynamic addgroup and adduser line to add the user/group associated with the env values, manually returns 0 on that line to always return successful (e.g. when using user root or www-data). Adds sed lines to change the group and user associated with the php service, removes the conf file originally overwriting those. Adds CMD line to start php-fpm with the -R flag, allowing root user.
diff --git a/php.dockerfile b/php.dockerfile
@@ -1,18 +1,25 @@
 FROM php:7.4-fpm-alpine
 
-ADD ./php/www.conf /usr/local/etc/php-fpm.d/
+ARG PHPGROUP
+ARG PHPUSER
 
-RUN addgroup -g 1000 laravel && adduser -G laravel -g laravel -s /bin/sh -D laravel
+ENV PHPGROUP=${PHPGROUP}
+ENV PHPUSER=${PHPUSER}
 
-RUN mkdir -p /var/www/html
+RUN addgroup -g 1000 ${PHPGROUP} && adduser -g ${PHPGROUP} -s /bin/sh -D ${PHPUSER}; exit 0
 
-RUN chown laravel:laravel /var/www/html
+RUN mkdir -p /var/www/html
 
 WORKDIR /var/www/html
 
+RUN sed -i "s/user = www-data/user = ${PHPUSER}/g" /usr/local/etc/php-fpm.d/www.conf
+RUN sed -i "s/group = www-data/group = ${PHPGROUP}/g" /usr/local/etc/php-fpm.d/www.conf
+
 RUN docker-php-ext-install pdo pdo_mysql
 
 RUN mkdir -p /usr/src/php/ext/redis \
     && curl -L https://github.com/phpredis/phpredis/archive/5.3.4.tar.gz | tar xvz -C /usr/src/php/ext/redis --strip 1 \
     && echo 'redis' >> /usr/src/php-available-exts \
     && docker-php-ext-install redis
+
+CMD ["php-fpm", "-y", "/usr/local/etc/php-fpm.conf", "-R"]
