Merge pull request #42198 from nextcloud/artonge/fix/metadata_error

Wrap metadata generation in try/catch

Author: artonge

core/BackgroundJobs/GenerateMetadataJob.php

@@ -33,6 +33,7 @@
 use OCP\FilesMetadata\IFilesMetadataManager;
 use OCP\IConfig;
 use OCP\IUserManager;
+use Psr\Log\LoggerInterface;
 
 class GenerateMetadataJob extends TimedJob {
 	public function __construct(
@@ -42,6 +43,7 @@ public function __construct(
 		private IUserManager $userManager,
 		private IFilesMetadataManager $filesMetadataManager,
 		private IJobList $jobList,
+		private LoggerInterface $logger,
 	) {
 		parent::__construct($time);
 
@@ -102,10 +104,14 @@ private function scanFolder(Folder $folder): void {
 				continue;
 			}
 
-			$this->filesMetadataManager->refreshMetadata(
-				$node,
-				IFilesMetadataManager::PROCESS_LIVE | IFilesMetadataManager::PROCESS_BACKGROUND
-			);
+			try {
+				$this->filesMetadataManager->refreshMetadata(
+					$node,
+					IFilesMetadataManager::PROCESS_LIVE | IFilesMetadataManager::PROCESS_BACKGROUND
+				);
+			} catch (\Throwable $ex) {
+				$this->logger->warning("Error while generating metadata for fileid ".$node->getId(), ['exception' => $ex]);
+			}
 		}
 	}
 }