HEAD returns a 301 for exists function when directory exists about easywebdav HOT 6 CLOSED

I could force the method _send to accept 301:

 self._send('HEAD', remote_path, (200, 301, 404))

But I'm not sure if it is the right way / coherent with the webdav procotol.

from easywebdav.

I'm also unsure...

from easywebdav.

It seems the '301' case is managed by the ls and mkdir commands. Maybe factorize the code about this case then extends exists for it ?

from easywebdav.

usually redirection could be manage by requests. Why not put a switch to activate it globaly and/or for each function as optional parameter ? The programmer will choose if it is acceptable to redirect or not.

from easywebdav.

Well @kartoch, I'm not sure WebDAV 301 should actually be treated as a redirection. The protocol is a bit unclear to me, to be honest. I just change and fix stuff according to comments of developers who use this library.

So anyway, yes - we can just add this code to the expected codes I guess. I'm personally against turning on automatic redirections in requests because I'm not sure whether or not it makes sense. If you try that yourself and find that it works as expected, please let me know what you tested and how, and submit a pull request.

from easywebdav.

Added 301 to expected result of exists().

from easywebdav.