diff options
Diffstat (limited to 'core/BackgroundJobs')
4 files changed, 23 insertions, 9 deletions
diff --git a/core/BackgroundJobs/BackgroundCleanupUpdaterBackupsJob.php b/core/BackgroundJobs/BackgroundCleanupUpdaterBackupsJob.php index 231caa683af..d1ecc08ca4b 100644 --- a/core/BackgroundJobs/BackgroundCleanupUpdaterBackupsJob.php +++ b/core/BackgroundJobs/BackgroundCleanupUpdaterBackupsJob.php @@ -10,6 +10,7 @@ namespace OC\Core\BackgroundJobs; use OCP\AppFramework\Utility\ITimeFactory; use OCP\BackgroundJob\QueuedJob; +use OCP\Files; use OCP\IConfig; use Psr\Log\LoggerInterface; @@ -28,17 +29,20 @@ class BackgroundCleanupUpdaterBackupsJob extends QueuedJob { * @param array $argument */ public function run($argument): void { + $this->log->info('Running background job to clean-up outdated updater backups'); + $updateDir = $this->config->getSystemValue('updatedirectory', null) ?? $this->config->getSystemValue('datadirectory', \OC::$SERVERROOT . '/data'); $instanceId = $this->config->getSystemValue('instanceid', null); if (!is_string($instanceId) || empty($instanceId)) { + $this->log->error('Skipping updater backup clean-up - instanceId is missing!'); return; } $updaterFolderPath = $updateDir . '/updater-' . $instanceId; $backupFolderPath = $updaterFolderPath . '/backups'; if (file_exists($backupFolderPath)) { - $this->log->info("$backupFolderPath exists - start to clean it up"); + $this->log->debug("Updater backup folder detected: $backupFolderPath"); $dirList = []; $dirs = new \DirectoryIterator($backupFolderPath); @@ -52,6 +56,8 @@ class BackgroundCleanupUpdaterBackupsJob extends QueuedJob { $realPath = $dir->getRealPath(); if ($realPath === false) { + $pathName = $dir->getPathname(); + $this->log->warning("Skipping updater backup folder: $pathName (not found)"); continue; } @@ -61,15 +67,18 @@ class BackgroundCleanupUpdaterBackupsJob extends QueuedJob { ksort($dirList); // drop the newest 3 directories $dirList = array_slice($dirList, 0, -3); - $this->log->info('List of all directories that will be deleted: ' . json_encode($dirList)); + $this->log->debug('Updater backup folders that will be deleted: ' . json_encode($dirList)); foreach ($dirList as $dir) { $this->log->info("Removing $dir ..."); - \OC_Helper::rmdirr($dir); + $result = Files::rmdirr($dir); + if (!$result) { + $this->log->error('Could not remove updater backup folder $dir'); + } } - $this->log->info('Cleanup finished'); + $this->log->info('Background job to clean-up updater backups has finished'); } else { - $this->log->info("Could not find updater directory $backupFolderPath - cleanup step not needed"); + $this->log->warning("Skipping updater backup clean-up - could not find updater backup folder $backupFolderPath"); } } } diff --git a/core/BackgroundJobs/CheckForUserCertificates.php b/core/BackgroundJobs/CheckForUserCertificates.php index 7fa5951d775..c4ac28905ef 100644 --- a/core/BackgroundJobs/CheckForUserCertificates.php +++ b/core/BackgroundJobs/CheckForUserCertificates.php @@ -32,7 +32,7 @@ class CheckForUserCertificates extends QueuedJob { */ public function run($arguments): void { $uploadList = []; - $this->userManager->callForSeenUsers(function (IUser $user) use (&$uploadList) { + $this->userManager->callForSeenUsers(function (IUser $user) use (&$uploadList): void { $userId = $user->getUID(); try { \OC_Util::setupFS($userId); diff --git a/core/BackgroundJobs/GenerateMetadataJob.php b/core/BackgroundJobs/GenerateMetadataJob.php index e775717092a..cb02a8e4ffa 100644 --- a/core/BackgroundJobs/GenerateMetadataJob.php +++ b/core/BackgroundJobs/GenerateMetadataJob.php @@ -8,6 +8,7 @@ declare(strict_types=1); namespace OC\Core\BackgroundJobs; +use OC\Files\Mount\MoveableMount; use OCP\AppFramework\Utility\ITimeFactory; use OCP\BackgroundJob\IJobList; use OCP\BackgroundJob\TimedJob; @@ -83,7 +84,7 @@ class GenerateMetadataJob extends TimedJob { private function scanFolder(Folder $folder): void { // Do not scan share and other moveable mounts. - if ($folder->getMountPoint() instanceof \OC\Files\Mount\MoveableMount) { + if ($folder->getMountPoint() instanceof MoveableMount) { return; } diff --git a/core/BackgroundJobs/LookupServerSendCheckBackgroundJob.php b/core/BackgroundJobs/LookupServerSendCheckBackgroundJob.php index 01eec5b3ce1..86622e58758 100644 --- a/core/BackgroundJobs/LookupServerSendCheckBackgroundJob.php +++ b/core/BackgroundJobs/LookupServerSendCheckBackgroundJob.php @@ -27,8 +27,12 @@ class LookupServerSendCheckBackgroundJob extends QueuedJob { * @param array $argument */ public function run($argument): void { - $this->userManager->callForSeenUsers(function (IUser $user) { - $this->config->setUserValue($user->getUID(), 'lookup_server_connector', 'dataSend', '1'); + $this->userManager->callForSeenUsers(function (IUser $user): void { + // If the user data was not updated yet (check if LUS is enabled and if then update on LUS or delete on LUS) + // then we need to flag the user data to be checked + if ($this->config->getUserValue($user->getUID(), 'lookup_server_connector', 'dataSend', '') === '') { + $this->config->setUserValue($user->getUID(), 'lookup_server_connector', 'dataSend', '1'); + } }); } } |