Skip to content
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.

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

Fix permissions on mounted settings #28

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Lxtharia
Copy link

@Lxtharia Lxtharia commented Apr 4, 2024

I wanted to mount my settings.php, but the sed -i commands in run.sh do not work on "busy files", so this uses a temporary file which will be written to the mounted file after it is edited.
But because the sed -i commands were also responsible of changing the permissions of the files from ownership root to www-data1, we also need to correct the permissions in the Dockerfile

Footnotes

  1. Took me a while to debug this. But apparently, if you have a file owned by root in a directory owned by you, you can remove it. This is what sed -i essentially does, but it looks like it "changed ownership".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant