Skip to content

Commit d0916e3

Browse files
authored
Merge pull request #55985 from nextcloud/backport/55529/stable31
[stable31] fix: make share recursion error more informative
2 parents 14a9c71 + a09715c commit d0916e3

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

apps/files_sharing/lib/SharedStorage.php

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -134,8 +134,9 @@ private function init() {
134134
// this is probably because some code path has caused recursion during the share setup
135135
// we setup a "failed storage" so `getWrapperStorage` doesn't return null.
136136
// If the share setup completes after this the "failed storage" will be overwritten by the correct one
137-
$this->logger->warning('Possible share setup recursion detected');
138-
$this->storage = new FailedStorage(['exception' => new \Exception('Possible share setup recursion detected')]);
137+
$ex = new \Exception('Possible share setup recursion detected for share ' . $this->superShare->getId());
138+
$this->logger->warning($ex->getMessage(), ['exception' => $ex, 'app' => 'files_sharing']);
139+
$this->storage = new FailedStorage(['exception' => $ex]);
139140
$this->cache = new FailedCache();
140141
$this->rootPath = '';
141142
}

0 commit comments

Comments
 (0)