Merge pull request #22561 from nextcloud/backport/22556/stable18

[stable18] Do not keep the part file if the forbidden exception has no retry set
nextcloud · Sep 3, 2020 · d663117 · d663117
2 parents 300f571 + 163125f
commit d663117
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/apps/dav/lib/Connector/Sabre/File.php b/apps/dav/lib/Connector/Sabre/File.php
@@ -280,6 +280,9 @@ public function put($data) {
 						throw new Exception('Could not rename part file to final file');
 					}
 				} catch (ForbiddenException $ex) {
+					if (!$ex->getRetry()) {
+						$partStorage->unlink($internalPartPath);
+					}
 					throw new DAVForbiddenException($ex->getMessage(), $ex->getRetry());
 				} catch (\Exception $e) {
 					$partStorage->unlink($internalPartPath);