Skip to content

[13] only allow a single concurrent dav write to a file - #9392

Merged
rullzer merged 1 commit into
stable13from
dav-upload-lock-13
May 23, 2018
Merged

[13] only allow a single concurrent dav write to a file#9392
rullzer merged 1 commit into
stable13from
dav-upload-lock-13

Conversation

@icewind1991

Copy link
Copy Markdown
Member

Backport of #9355

Signed-off-by: Robin Appelman <robin@icewind.nl>
@icewind1991icewind1991 added the 3. to review Waiting for reviews label May 4, 2018
@icewind1991icewind1991 added this to the Nextcloud 13.0.3 milestone May 4, 2018
$result = $node->put($data);

$this->fileView->unlockFile($path . '.upload.part', ILockingProvider::LOCK_EXCLUSIVE);
$node->releaseLock(ILockingProvider::LOCK_SHARED);

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is also new?

@rullzer
rullzer merged commit 56b8a88 into stable13May 23, 2018
@rullzer
rullzer deleted the dav-upload-lock-13 branch May 23, 2018 18:14
@MorrisJobkeMorrisJobke mentioned this pull request May 31, 2018
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@icewind1991@rullzer@nickvergessen@MorrisJobke