Merge pull request #315 from magikmw/master

Fix requirement for using root in docker container to use port 80
ondras · Aug 3, 2022 · 8951e68 · 8951e68
2 parents 4c2b6f5 + 4a5bb13
commit 8951e68
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/Dockerfile b/Dockerfile
@@ -1,6 +1,10 @@
 FROM php:5-apache
 
-EXPOSE 80
+RUN sed -i 's/Listen 80/Listen ${PORT}/' /etc/apache2/ports.conf
+
+ENV PORT=8080
+
+EXPOSE 8080
 
 WORKDIR /var/www/html
 

diff --git a/README.md b/README.md
@@ -24,7 +24,7 @@ If you wish to support this project, <a href='https://www.paypal.com/cgi-bin/web
 ## Docker Installation:
 
 1. Build `docker build -t wwwsqldesigner .`
-2. Run   `docker run -d -p 8080:80 wwwsqldesigner`
+2. Run   `docker run -d -p 8080:8080 wwwsqldesigner`
 3. Visit http://127.0.0.1:8080
 
 # News