mirror/librenms-librenms
1
0
Fork 0
mirror of https://github.com/librenms/librenms.git synced 2024-10-07 16:52:45 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Drop group write validation (#11873)

Browse Source 
causes more pain than it highlights issues
New install docs mitigate the original issue
This commit is contained in:
Tony Murray
2020-06-29 09:29:43 -05:00
committed by GitHub
parent 9de48cfae2
commit 63ba612fe8
1 changed files with 0 additions and 19 deletions
Download Patch File Download Diff File Expand all files Collapse all files

19
LibreNMS/Validations/User.php
View File

@@ -118,25 +118,6 @@ class User extends BaseValidation
return;
}
}
// check folder permissions
$folders = [
'rrd' => $rrd_dir,
'log' => $log_dir,
'bootstrap' => "$dir/bootstrap/cache/",
'storage' => "$dir/storage/",
'cache' => "$dir/storage/framework/cache/",
'sessions' => "$dir/storage/framework/sessions/",
'views' => "$dir/storage/framework/views/",
];
$folders_string = implode(' ', $folders);
$incorrect = exec("find $folders_string -group $lnms_groupname ! -perm -g=w");
if (!empty($incorrect)) {
$validator->result(ValidationResult::fail(
'Some folders have incorrect file permissions, this may cause issues.'
)->setFix($fix)->setList('Files', explode(PHP_EOL, $incorrect)));
}
} else {
$validator->warn("You don't have \$config['user'] set, this most likely needs to be set to librenms");
}