feat(templates): set ownership and permissions on private files folder #82
Conversation
| chown -R ${drupal_user}:${httpd_group} ${private_files_path} & | ||
| printf "Changing permissions of all files inside private files directory ${private_files_path} to rw-rw----...\n" && \ | ||
| printf "Changing permissions of private files directory and all directories inside ${private_files_path} to rwxrwx---...\n" && \ | ||
| chmod u=rwx,g=rwx,o= ${private_files_path} |
There was a problem hiding this comment.
Doesn't this line (248) become redundant if the two 'find' commands below apply the correct permissions to the same path?
There was a problem hiding this comment.
@finalhow I wasn't sure if the find command would apply the permissions to the containing (top-level) directory, or if the find command would only return and apply permissions to only the directories contained within that top-level private files directory.
There was a problem hiding this comment.
Ah makes sense. Well, either way its just one chmod extra on one directory and its not recursive so no big deal.
Easy way to test if ever you need to: Run the same find command without the 'exec' bit and 'head' the output:
find <path> -type d | head
Anyway ok perfect.
There was a problem hiding this comment.
@ClassicCut Forgot to @ mention my reply... bah
|
@finalhow Added an extra check for ensuring nothing gets set on the root folder, in case the variable gets that value somehow. |
#80