Skip to content

Commit ea64fd9

Browse files
Merge pull request #62226 from nextcloud/backport/62208/stable34
[stable34] fix: don't die if we can't encode file metadata on upload
2 parents ddee65a + ed9049a commit ea64fd9

1 file changed

Lines changed: 4 additions & 0 deletions

File tree

lib/private/FilesMetadata/FilesMetadataManager.php

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -164,6 +164,10 @@ public function saveMetadata(IFilesMetadata $filesMetadata): void {
164164
}
165165

166166
$json = json_encode($filesMetadata->jsonSerialize());
167+
if (!$json) {
168+
$this->logger->error('Failed to json encode file metadata for ' . $filesMetadata->getFileId(), ['metadata' => $filesMetadata->jsonSerialize()]);
169+
return;
170+
}
167171
if (strlen($json) > self::JSON_MAXSIZE) {
168172
$this->logger->debug('huge metadata content detected: ' . $json);
169173
throw new FilesMetadataException('json cannot exceed ' . self::JSON_MAXSIZE . ' characters long; fileId: ' . $filesMetadata->getFileId() . '; size: ' . strlen($json));

0 commit comments

Comments
 (0)