Merge pull request #6 from chschu/master

remove shells to make nginx the main process of the container
schmunk42 · Feb 11, 2018 · bd998ed · bd998ed
2 parents 02a34b8 + 048a46f
commit bd998ed
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/Dockerfile b/Dockerfile
@@ -3,4 +3,6 @@ FROM nginx:alpine
 ADD run.sh /run.sh
 ADD default.conf /etc/nginx/conf.d/default.conf
 
-CMD sh run.sh
+RUN chmod +x /run.sh
+
+CMD ["/run.sh"]
diff --git a/run.sh b/run.sh
@@ -25,4 +25,4 @@ sed -i "s|\${SERVER_REDIRECT_CODE}|${SERVER_REDIRECT_CODE}|" /etc/nginx/conf.d/d
 sed -i "s|\${SERVER_REDIRECT_PATH}|${SERVER_REDIRECT_PATH}|" /etc/nginx/conf.d/default.conf
 sed -i "s|\${SERVER_REDIRECT_SCHEME}|${SERVER_REDIRECT_SCHEME}|" /etc/nginx/conf.d/default.conf
 
-nginx -g 'daemon off;'
+exec nginx -g 'daemon off;'