Skip to content

Auto chmod for /data/www #8

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

Closed
wants to merge 1 commit into from
Closed

Auto chmod for /data/www #8

wants to merge 1 commit into from

Conversation

smyth64
Copy link

@smyth64 smyth64 commented Jan 8, 2016

I faced a problem when I was syncing my files from my volume to /data/www and they always had the root:root owner. This little supervisor script watches all files and automatically changes the owner to www:www.

If you think this will cause any performance issues please give me some suggestion for improvement.

Best regards,
Michael

I faced a problem when I was syncing my files from my volume to /data/www and they always had the root:root owner. This little supervisor script watches all files and automatically changes the owner to www:www.

If you think this will cause any performance issues please give me some suggestion for improvement.


Best regards,
Michael
@ryzy
Copy link
Member

ryzy commented Feb 10, 2016

Hi Smithi,

Sorry, thought I replied to that PR... but must have missed it.

Performance issues aside, it would be pretty risky to assume that everyone on each project wants to have by default such permissions. Some apps might set more strict permissions and you don't want to relax them by default.

My suggestion is to include it as part of your syncing process.

@ryzy ryzy closed this Feb 10, 2016
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.

2 participants