Fix: Resolve incompatibility issues for current environment - Amine #24

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Open

smaoui-me wants to merge 1 commit into t03i:main from smaoui-me:compatibility-fix_amine

smaoui-me commented Nov 4, 2025

backend-entrypoint.sh: Replaced the unconditional recursive chown -R user:usergroup /app with a conditional, iterative check. .

The original chown -R would cause the container to fail to start when using set -e if any part of the /app volume (e.g., a critical config file or a whole volume) was mounted as read-only.
The new logic checks for write permissions (-w) on the /app root and then on each item inside it. It only executes chown -R on writable subdirectories/files, skipping read-only paths and logging a message instead.

docker-compose.dev.yaml: - Mounting the entire ./backend/data directory as rw simplifies volume management and ensures the new script logic can correctly handle permissions inside /app/data (where the database lives). This has resolved the mounting errors, I've encountered.

Exposed port 8000 to the host.

Exposed port 8000 to the host.


          Fix: Resolve incompatibility issues for current environment - Amine

43aaefa

smaoui-me marked this pull request as ready for review

November 4, 2025 11:05

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet