Skip to content

Return 204 status if no locks are present - #1672

Open
mschering wants to merge 1 commit into
sabre-io:masterfrom
mschering:patch-1
Open

mschering wants to merge 1 commit into
sabre-io:masterfrom
mschering:patch-1

Conversation

@mschering

Copy link
Copy Markdown

I ran into an issue with Mountain Duck and file locking. When I opened a file it locks it as expected. I left the file open for a day and then the next day the lock has expired. MD keeps sending that expired lock token and there was no way to re-edit the file. Even when reconnecting the webdav drive. I figured that the UNLOCK request does not necessarily have to fail if the file is not locked at all. So these lines of code fixed the issue for me. I hope you can merge this!

I ran into an issue with Mountain Duck and file locking. When I opened a file it locks it as expected. I left the file open for a day and then the next day the lock has expired.
MD keeps sending that expired lock token and there was no way to re-edit the file. Even when reconnecting the webdav drive.
I figured that the UNLOCK request does not necessarily have to fail if the file is not locked at all. So these lines of code fixed the issue for me. I hope you can merge this!
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant