Skip to content

Conversation

@flymarq
Copy link

@flymarq flymarq commented Jul 25, 2025

The Uri is derived from the request with the method getPath, which generates a valid path fragment suitable for file and directory names.

In the XML-response to a lock-request the uri is used for lockroot element. If the uri contains a space or german umlauts, a strict client like libneon, which is used in davfs2, refuses the lock reponse and files cannot be copied.

flymarq added 2 commits July 25, 2025 12:26
The Uri is derived from the request with the method getPath, which generates a valid path fragment suitable for file and directory names.

In the XML-response to a lock-request the uri is used for lockroot element. If the uri contains a space or german umlauts, a strict client like libneon, which is used in davfs2, refuses the lock reponse and files cannot be copied.
- allow upload of multipe files
- uploads are not limited by php upload size using PUT
- progress bar for each file
@alecpl
Copy link

alecpl commented Nov 21, 2025

I see the second commit here is unrelated and should be removed. Also, it would be great to have a test added in tests/Sabre/DAV/Locks/PluginTest.php.

@flymarq
Copy link
Author

flymarq commented Nov 21, 2025

My fault, the second commit does not belong to this issue.

I can try to add a test, but I'm afraid it won't work unless a strict client like libneon performs the test.

@flymarq
Copy link
Author

flymarq commented Nov 21, 2025

Done:

  • remove second commit
  • add LOCK test on url with space with expected http status value 201

Before the patch we received a http status value 423.

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.

2 participants