From 23b62023c0a911d6f0b4438f75d207043dd47fed Mon Sep 17 00:00:00 2001 From: Simon Rettberg Date: Wed, 28 Feb 2024 09:44:37 +0100 Subject: [backup] Checking file existence is stupid if we write to /root/ php doesn't run as root and doesn't have access to that location. --- modules-available/backup/hooks/cron.inc.php | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/modules-available/backup/hooks/cron.inc.php b/modules-available/backup/hooks/cron.inc.php index 3117ad36..c4a4b1dc 100644 --- a/modules-available/backup/hooks/cron.inc.php +++ b/modules-available/backup/hooks/cron.inc.php @@ -31,14 +31,10 @@ EventLog::failure("Could not create automatic backup, Taskmanager down"); return; } - $task = Taskmanager::waitComplete($task, 60000); + $task = Taskmanager::waitComplete($task, 90000); if (!Taskmanager::isFinished($task) || Taskmanager::isFailed($task) || !isset($task['data']['backupFile'])) { EventLog::failure("Creating backup failed", print_r($task, true)); return; } - if (!file_exists($task['data']['backupFile'])) { - EventLog::failure("Backup file '{$task['data']['backupFile']}' not found after successful backup run"); - return; - } Property::set(BackupRestore::PROP_LAST_BACKUP, time()); })(); \ No newline at end of file -- cgit v1.2.3-55-g7522