fix: ensure storage path permissions are set in entry script

docker/Dockerfile

@@ -7,15 +7,12 @@ ENV APP_DEBUG=true
 ENV SELF_HOSTED=true
 
 # Set the working directory
-COPY . /var/www/app
-WORKDIR /var/www/app
-
-# Allow PHP installs to be built cross-platform
-#ENV CFLAGS="-fstack-protector-strong -fpic -fPIC -fpie -O2 -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64"
+COPY . /var/app
+WORKDIR /var/app
 
 # Install required packages
 RUN apt-get update && apt-get upgrade -y \
-    && apt-get upgrade -y \
+    && apt-get install -y \
         nginx \
         libfreetype-dev \
         libjpeg62-turbo-dev \
@@ -37,20 +34,22 @@ RUN apt-get update && apt-get upgrade -y \
     && docker-php-ext-install -j$(nproc) \
         gd pgsql zip pdo_mysql mysqli intl
 
-# Set permissions
+# Remove default nginx config
+RUN rm /etc/nginx/sites-enabled/default \
+    && rm -rf /var/www/html \
+    && ln -s /var/app /var/www/app
+
+# Set permissions and ensure www-data has a shell available
 RUN chown -R www-data:www-data . \ 
     && chmod -R 775 ./storage \
     && chmod +x ./docker/entrypoint.sh \
-    && usermod -s /bin/bash www-data
+    && usermod -s /bin/bash www-data 
 
 # Install Composer and Node.js Install PHP dependencies and build front end assets
 RUN curl -sS https://getcomposer.org/installer | php -- --install-dir=/usr/local/bin --filename=composer \
     && composer install --no-scripts --optimize-autoloader \
     && npm install && npm run build
 
-# Remove default nginx config
-RUN rm /etc/nginx/sites-enabled/default
-
 # Copy over configs
 COPY ./docker/nginx.conf /etc/nginx/conf.d/default.conf
 COPY ./docker/supervisord.conf /etc/supervisor/conf.d/supervisord.conf

docker/entrypoint.sh

@@ -1,27 +1,8 @@
 #!/bin/bash
 
-cd /var/www/app
+cd /var/app
 
 echo -e "\n====================== Validating environment...  ====================== "
-if [[ -z "$APP_KEY" ]]; then
-    echo -e "\n > Oops! The required APP_KEY configuration is missing in your environment! "
-    echo -e "\n > Generating a key (see below) but this will NOT be persisted between container restarts. "
-    echo -e "\n > You should set this APP_KEY in your .env file! "
-
-    draw_box() {
-      local text="$1"
-      local length=${#text}
-      local border=$(printf '%*s' "$((length + 4))" | tr ' ' '*')
-
-      echo -e "\n\n$border"
-      echo "* $text *"
-      echo "$border"
-    }
-
-    export APP_KEY=base64:$(openssl rand -base64 32)
-    draw_box $APP_KEY
-fi
-
 for dir in storage/framework/cache storage/framework/sessions storage/framework/views; do
     if [ ! -d "$dir" ]; then
         echo -e "\n > $dir is missing. Creating scaffold for storage directory... "
@@ -34,14 +15,32 @@ done
 if [ ! -L "public/storage" ]; then
     echo -e "\n > Creating symbolic link for app public storage... "
     
-    /usr/local/bin/php /var/www/app/artisan storage:link
+    php artisan storage:link
+fi
+
+if [[ -z "$APP_KEY" ]]; then
+    echo -e "\n > Oops! The required APP_KEY configuration is missing in your environment! "
+    echo -e "\n > You should set this APP_KEY in your .env file! "
+
+    draw_box() {
+      local text="$1"
+      local length=${#text}
+      local border=$(printf '%*s' "$((length + 4))" | tr ' ' '*')
+
+      echo -e "\n\n$border"
+      echo "* $text *"
+      echo "$border"
+    }
+
+    export APP_KEY=$(php artisan key:generate --show)
+    draw_box $APP_KEY
 fi
 
 echo -e "\n====================== Running migrations...  ====================== "
 run_migrations() {
-    /usr/local/bin/php /var/www/app/artisan migrate --force
+    php artisan migrate --force
 }
-RETRIES=10
+RETRIES=12 # wait 60 seconds for database to be ready
 DELAY=5
 until run_migrations; do
   RETRIES=$((RETRIES-1))
@@ -54,4 +53,4 @@ until run_migrations; do
 done
 
 echo -e "\n====================== Spinning up Supervisor daemon...  ====================== \n"
-exec /usr/bin/supervisord -c /etc/supervisor/conf.d/supervisord.conf
+exec supervisord -c /etc/supervisor/conf.d/supervisord.conf

docker/nginx.conf

@@ -2,7 +2,7 @@ server {
     listen 80;
     server_name _;
 
-    root /var/www/app/public;
+    root /var/app/public;
     index index.php index.html index.htm;
 
     location / {