From 6783dc1312f52f2578ea5d556b68f61371b96afe Mon Sep 17 00:00:00 2001 From: Lee Porte Date: Wed, 22 May 2024 16:24:52 +0100 Subject: [PATCH] Fixed #12299: permissions on storage dir in Docker Fixes #12299, where currently there is no recursive permissions set on /var/www/html/storage/ during build time. As a result this requires users to run an exec into the container and change the permissions before they are able to initiate the setup. Without this a 500 error is thrown and little is evident in the docker logs as to what the issue is. --- Dockerfile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Dockerfile b/Dockerfile index 88de52858b..bd363ccd18 100644 --- a/Dockerfile +++ b/Dockerfile @@ -105,7 +105,7 @@ RUN \ && ln -fs "/var/lib/snipeit/keys/ldap_client_tls.cert" "/var/www/html/storage/ldap_client_tls.cert" \ && ln -fs "/var/lib/snipeit/keys/ldap_client_tls.key" "/var/www/html/storage/ldap_client_tls.key" \ && chown docker "/var/lib/snipeit/keys/" \ - && chown -h docker "/var/www/html/storage/" \ + && chown -Rh docker "/var/www/html/storage/" \ && chmod +x /var/www/html/artisan \ && echo "Finished setting up application in /var/www/html"