Merge pull request #22560 from nextcloud/backport/22556/stable19

[stable19] Do not keep the part file if the forbidden exception has no retry set
nextcloud · Sep 3, 2020 · 468f1fc · 468f1fc
2 parents 45dcdc3 + f4dfb96
commit 468f1fc
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
@@ -288,6 +288,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);