Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

please add lock feature #8

Closed
ifooth opened this issue Apr 22, 2013 · 14 comments

Comments

@ifooth
Copy link

@ifooth ifooth commented Apr 22, 2013

please add lock feature!
webdav class 2

@arut

This comment has been minimized.

Copy link
Owner

@arut arut commented Apr 22, 2013

I will add when I have enough time. I'm now very busy.
Or maybe someone pays for that.

@alandekok

This comment has been minimized.

Copy link

@alandekok alandekok commented Sep 18, 2013

How much?

@mikeryan

This comment has been minimized.

Copy link

@mikeryan mikeryan commented Dec 10, 2013

Also willing to contribute cash to the effort.

@markc

This comment has been minimized.

Copy link

@markc markc commented Jan 10, 2014

Or maybe someone pays for that.

So, arut, how much would you like to be paid for implementing full webdav 2 support?

ATM I still have to use lighttpd for webdav.

@anatoli26

This comment has been minimized.

Copy link

@anatoli26 anatoli26 commented Jan 8, 2017

Hi @arut, I'm also willing to contribute economically to the full webdav support with locks. Could you please let us know how much is required so we collect needed funds on bountysource.com or similar platform? The problem is that without locks there's no write access on OS X, so the whole thing is quite unusable.

@grigory51

This comment has been minimized.

Copy link

@grigory51 grigory51 commented Mar 7, 2017

+1

@anatoli26

This comment has been minimized.

Copy link

@anatoli26 anatoli26 commented Jul 25, 2017

Hi @arut, I see some changes in the repo. Could you please consider adding this feature? This is IMO the most important feature lacking as its absence makes impossible using this extension with a number of webdav client implementations, in particular MacOS.

@alehaa

This comment has been minimized.

Copy link

@alehaa alehaa commented Apr 20, 2018

Does someone know, if there's a easy to read documentation on the lock feature, so maybe we could write a patch for this feature?

@anatoli26

This comment has been minimized.

Copy link

@anatoli26 anatoli26 commented Apr 21, 2018

AFAIK, this is not a simple feature if you don't know well the NginX internals, as it requires to maintain a DB of locks. I suggest you to check the implementation details of Cyrus IMAP (the HTTP module and xDAV features): https://github.com/cyrusimap/cyrus-imapd/blob/master/imap/http_dav.c.

By the way, if you deploy MDA (Mail Delivery Agent) in your organization (like Dovecot, etc.) I'd recommend you to migrate to Cyrus and have 2-in-1: MDA + xDAV (CalDAV, CardDAV and WebDAV). It works well with all devices and is super easy to configure. This is what we've finally implemented and we're completely satisfied with the result.

@alehaa

This comment has been minimized.

Copy link

@alehaa alehaa commented Dec 4, 2018

Unfortunately I didn't find an easy solution to implement this feature, so I'll pass through all WebDAV requests (non GET, POST, HEAD) to a python WebDAV server now. This is not the solution I was hoping to find, but it works.

@JoeKun

This comment has been minimized.

Copy link

@JoeKun JoeKun commented Dec 4, 2018

@alehaa Would you mind telling us which python WebDAV server you used?

@alehaa

This comment has been minimized.

Copy link

@alehaa alehaa commented Dec 5, 2018

@JoeKun I use the python-webdav package from debian to spawn a WebDAV server at localhost.

@arut

This comment has been minimized.

Copy link
Owner

@arut arut commented Dec 6, 2018

Guys, I've just pushed a branch v2locks with locking support. It currently works at least with the MacOS client. Any feedback is highly appreciated.

@arut

This comment has been minimized.

Copy link
Owner

@arut arut commented Dec 17, 2018

Locking added.

@arut arut closed this Dec 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
9 participants
You can’t perform that action at this time.