From 5735feec3a5834a45b8cc91ba1dbc312ed6c218b Mon Sep 17 00:00:00 2001 From: Wilmer Arambula Date: Fri, 11 Jul 2025 16:51:46 -0400 Subject: [PATCH 1/2] feat: Add ownership and permissions setup for node_modules directory in initialization script. --- docker/entrypoint.sh | 2 +- docker/init.sh | 8 ++++++++ 2 files changed, 9 insertions(+), 1 deletion(-) diff --git a/docker/entrypoint.sh b/docker/entrypoint.sh index c22af76..7ce7385 100644 --- a/docker/entrypoint.sh +++ b/docker/entrypoint.sh @@ -1,5 +1,5 @@ #!/bin/bash -+set -euo pipefail +set -euo pipefail echo "=== Container Starting ===" echo "Running initialization script..." diff --git a/docker/init.sh b/docker/init.sh index 6fd8671..8cb4572 100644 --- a/docker/init.sh +++ b/docker/init.sh @@ -62,6 +62,14 @@ if [ -f "/app/composer.json" ] && [ ! -d "/app/vendor" ]; then echo -e "${GREEN}✓ Vendor directory ownership set${NC}" fi + # Set permissions for node_modules directory if it exists + if chown -R www-data:www-data /app/node_modules 2>/dev/null; then + chmod -R 775 /app/node_modules + echo -e "${GREEN}✓ Node modules directory ownership set${NC}" + else + echo -e "${YELLOW}⚠ Node modules directory ownership could not be set (mounted volume?)${NC}" + fi + echo -e "${GREEN}✓ Composer dependencies installed successfully.${NC}" fi From b7479f2fdd79a9f97e4c64435814954c9065ad72 Mon Sep 17 00:00:00 2001 From: Wilmer Arambula Date: Fri, 11 Jul 2025 16:58:03 -0400 Subject: [PATCH 2/2] fix: Ensure proper ownership and permissions setup for node_modules directory after Composer installation. --- docker/init.sh | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/docker/init.sh b/docker/init.sh index 8cb4572..92d6507 100644 --- a/docker/init.sh +++ b/docker/init.sh @@ -62,17 +62,17 @@ if [ -f "/app/composer.json" ] && [ ! -d "/app/vendor" ]; then echo -e "${GREEN}✓ Vendor directory ownership set${NC}" fi - # Set permissions for node_modules directory if it exists - if chown -R www-data:www-data /app/node_modules 2>/dev/null; then - chmod -R 775 /app/node_modules - echo -e "${GREEN}✓ Node modules directory ownership set${NC}" - else - echo -e "${YELLOW}⚠ Node modules directory ownership could not be set (mounted volume?)${NC}" - fi - echo -e "${GREEN}✓ Composer dependencies installed successfully.${NC}" fi +# Set permissions for node_modules directory if it exists +if chown -R www-data:www-data /app/node_modules 2>/dev/null; then + chmod -R 775 /app/node_modules + echo -e "${GREEN}✓ Node modules directory ownership set${NC}" +else + echo -e "${YELLOW}⚠ Node modules directory ownership could not be set (mounted volume?)${NC}" +fi + echo -e "${GREEN}Starting supervisord...${NC}" # Start supervisord