From 7f1fb6554922257af03fe4960b8e899cca77ae80 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?C=C3=B4me=20Chilliet?= Date: Tue, 3 Jun 2025 09:35:23 +0200 Subject: [PATCH 1/2] fix(files_versions): Log failure to compute node path MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit To avoid hidden bugs, add some debug logging when a node path cannot be computed from a received event, to have more information for debugging Signed-off-by: Côme Chilliet --- .../lib/Listener/FileEventsListener.php | 18 ++++++++++++++++++ 1 file changed, 18 insertions(+) diff --git a/apps/files_versions/lib/Listener/FileEventsListener.php b/apps/files_versions/lib/Listener/FileEventsListener.php index 4b7385d6e2d29..4e25fff7b0178 100644 --- a/apps/files_versions/lib/Listener/FileEventsListener.php +++ b/apps/files_versions/lib/Listener/FileEventsListener.php @@ -402,6 +402,24 @@ private function getPathForNode(Node $node): ?string { } } + try { + $this->logger->debug('Failed to compute path for node', [ + 'node' => [ + 'path' => $node->getPath(), + 'owner' => $owner, + 'fileid' => $node->getId(), + 'size' => $node->getSize(), + 'mtime' => $node->getMTime(), + ] + ]); + } catch (NotFoundException) { + $this->logger->debug('Failed to compute path for node', [ + 'node' => [ + 'path' => $node->getPath(), + 'owner' => $owner, + ] + ]); + } return null; } } From 15c8b5ac58add6e3d6ecf662dac5727e26bb6cc9 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?C=C3=B4me=20Chilliet?= Date: Tue, 17 Jun 2025 16:25:45 +0200 Subject: [PATCH 2/2] chore: Use an if/else instead of the try/catch to make meaning more obvious MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: Côme Chilliet --- apps/files_versions/lib/Listener/FileEventsListener.php | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/apps/files_versions/lib/Listener/FileEventsListener.php b/apps/files_versions/lib/Listener/FileEventsListener.php index 4e25fff7b0178..2cc5a55eedb97 100644 --- a/apps/files_versions/lib/Listener/FileEventsListener.php +++ b/apps/files_versions/lib/Listener/FileEventsListener.php @@ -12,6 +12,7 @@ use OC\Files\Filesystem; use OC\Files\Mount\MoveableMount; use OC\Files\Node\NonExistingFile; +use OC\Files\Node\NonExistingFolder; use OC\Files\View; use OCA\Files_Versions\Storage; use OCA\Files_Versions\Versions\INeedSyncVersionBackend; @@ -402,7 +403,7 @@ private function getPathForNode(Node $node): ?string { } } - try { + if (!($node instanceof NonExistingFile) && !($node instanceof NonExistingFolder)) { $this->logger->debug('Failed to compute path for node', [ 'node' => [ 'path' => $node->getPath(), @@ -412,7 +413,7 @@ private function getPathForNode(Node $node): ?string { 'mtime' => $node->getMTime(), ] ]); - } catch (NotFoundException) { + } else { $this->logger->debug('Failed to compute path for node', [ 'node' => [ 'path' => $node->getPath(),