Skip to content
This repository has been archived by the owner on Oct 4, 2023. It is now read-only.

#50 #51

Closed
wants to merge 1 commit into from
Closed

#50 #51

wants to merge 1 commit into from

Conversation

dotupNET
Copy link

Please release ASAP.

Fixes #50

@vauvenal5
Copy link
Contributor

Hi!

I am not sure this is a good idea. At least in its current state. The path which you get currently from WebDavFile can be used again as input in ls for example. However, if you add decoding on the way out you will have to add encoding on the way in. Otherwise you can not simply reuse the path that you get from the library to query the next sub folder. And it feels strange if the library handles decoding for you but you have to handle encoding yourself.

Also tests are missing and this would be a major bump in the version since it is breaking apps which handle en-/decoding already.

But maybe @jld3103 has another view on this. :)

@provokateurin
Copy link
Owner

@vauvenal5 I agree with your points

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

WebDavFile with whitespace
3 participants