saltyorg / Sandbox

Saltbox Sandbox
GNU General Public License v3.0
68 stars 94 forks source link

nextcloud add check_data_directory_permissions when "nextcloud_skip_data_directory_permissions_check: true" specified #201

Closed hereisderek closed 1 year ago

hereisderek commented 1 year ago

update nextcloud to use flag nextcloud_skip_data_directory_permissions_check to skip data folder permission check

Description

This for adding 'check_data_directory_permissions' => true, into config/config.php of nextcloud, to bypass permission check error, which may not always be feasible to solve especially if mounted under rclone mount.

the original error reads:

Your data directory is readable by other users.
Please change the permissions to 0770 so that the directory cannot be listed by other users.

solution: link

How Has This Been Tested?

tested the following scenarios

hereisderek commented 1 year ago

i don't think the lint error was accurate?

saltydk commented 1 year ago

The linter isn't wrong, no. Also why did you randomly rename all the task names?

hereisderek commented 1 year ago

The linter isn't wrong, no. Also why did you randomly rename all the task names?

i had seen similiar changes in other places and i thought it would give a clearer output

saltydk commented 1 year ago

The linter isn't wrong, no. Also why did you randomly rename all the task names?

i had seen similiar changes in other places and i thought it would give a clearer output

Even if we did this type of thing in the base roles your formatting was all over the place.

saltydk commented 1 year ago

Anyway, I've made the edits and it should be fine now.