Minor refactor and cleanup

This commit is contained in:
nirpt
2022-08-28 11:29:20 +02:00
parent 1194259e6e
commit 97da7f0d4f
2 changed files with 17 additions and 14 deletions
+5 -5
View File
@@ -2,12 +2,12 @@ FROM php:7.4.21-apache
ENV APP_VERSION=1.0.0 ENV APP_VERSION=1.0.0
RUN apt-get update && \ RUN apt-get update && \
apt-get install -y less nano tini curl tar git zip unzip && \ apt-get install -y --no-install-recommends less nano tini curl tar git zip unzip && \
#echo "**** # To install networking tools for testing purpose ****" && \ #echo "**** # To install networking tools for testing purpose ****" && \
apt-get install -y iputils-ping dnsutils net-tools procps apt-get install -y --no-install-recommends iputils-ping dnsutils net-tools procps && \
#echo "**** cleanup ****" && \ #echo "**** cleanup ****" && \
#apt-get autoremove -y && \ apt-get autoremove -y && \
#apt-get clean -y apt-get clean -y
# curl -SL -o app.tar.gz "https://github.com/giandonatoinverso/PHP-Dynamic-Qr-code/archive/${APP_VERSION}.tar.gz" \ # curl -SL -o app.tar.gz "https://github.com/giandonatoinverso/PHP-Dynamic-Qr-code/archive/${APP_VERSION}.tar.gz" \
RUN curl -SL -o app.tar.gz "https://github.com/giandonatoinverso/PHP-Dynamic-Qr-code/archive/refs/heads/master.tar.gz" \ RUN curl -SL -o app.tar.gz "https://github.com/giandonatoinverso/PHP-Dynamic-Qr-code/archive/refs/heads/master.tar.gz" \
@@ -17,7 +17,7 @@ RUN curl -SL -o app.tar.gz "https://github.com/giandonatoinverso/PHP-Dynamic-Qr-
&& rm app.tar.gz && rm app.tar.gz
RUN apt-get update RUN apt-get update
RUN apt-get install -y libzip-dev libjpeg62-turbo-dev libpng-dev libfreetype6-dev RUN apt-get install -y --no-install-recommends libzip-dev libjpeg62-turbo-dev libpng-dev libfreetype6-dev
# install composer # install composer
RUN php -r "copy('https://getcomposer.org/installer', 'composer-setup.php');" RUN php -r "copy('https://getcomposer.org/installer', 'composer-setup.php');"
+4 -1
View File
@@ -1,4 +1,5 @@
### How to run this image ### How to run this image
- currently this image must be build locally.(in future can be pulled from docker hub) - currently this image must be build locally.(in future can be pulled from docker hub)
- create folder to mount inside docker container - create folder to mount inside docker container
- `mkdir -p ./data/mariadb ./data/app/config ./data/app/saved_qrcode` - `mkdir -p ./data/mariadb ./data/app/config ./data/app/saved_qrcode`
@@ -13,6 +14,7 @@
- password: superadmin - password: superadmin
### Roadmap ### Roadmap
- Make install process run via docker-compose. - Make install process run via docker-compose.
- Work is partially done. docker-entrypoint.sh is created.(currently disabled due to next steps are pending) - Work is partially done. docker-entrypoint.sh is created.(currently disabled due to next steps are pending)
- qrcode/install/database.php should be adjusted to create database without form submission. - qrcode/install/database.php should be adjusted to create database without form submission.
@@ -21,7 +23,8 @@
- Automate Build image and push to docker hub. - Automate Build image and push to docker hub.
- Audit security flaws and fix it if needed. - Audit security flaws and fix it if needed.
### some debug commands ### Some debug commands
- `docker-compose logs` - `docker-compose logs`
- `PUID="$(id -u)" PGID="$(id -g)" docker-compose config` - `PUID="$(id -u)" PGID="$(id -g)" docker-compose config`
- `ps -aef | grep apache2` - `ps -aef | grep apache2`