Merge pull request #55984 from nextcloud/backport/55529/stable32

[stable32] fix: make share recursion error more informative
pull/56071/head
Daniel 2025-10-28 21:16:49 +07:00 committed by GitHub
commit 2948acb488
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 3 additions and 2 deletions

@ -135,8 +135,9 @@ class SharedStorage extends Jail implements LegacyISharedStorage, ISharedStorage
// this is probably because some code path has caused recursion during the share setup
// we setup a "failed storage" so `getWrapperStorage` doesn't return null.
// If the share setup completes after this the "failed storage" will be overwritten by the correct one
$this->logger->warning('Possible share setup recursion detected');
$this->storage = new FailedStorage(['exception' => new \Exception('Possible share setup recursion detected')]);
$ex = new \Exception('Possible share setup recursion detected for share ' . $this->superShare->getId());
$this->logger->warning($ex->getMessage(), ['exception' => $ex, 'app' => 'files_sharing']);
$this->storage = new FailedStorage(['exception' => $ex]);
$this->cache = new FailedCache();
$this->rootPath = '';
}