Fix Webdav storage PROPFIND request

Remove additional "a:prop" in PROPFIND request to match RFC 4918 section 9.1.3. Added Content-Type header as the body is not a true multipart POST. Signed-off-by: Vincent Petry <PVince81@yahoo.fr>

Fix Webdav storage PROPFIND request
687788e4 · Vincent Petry · 5348f8c9 · 687788e4
Unverified Commit 687788e4 authored Jan 04, 2021 by Vincent Petry
Hide whitespace changes
Inline Side-by-side

Showing with 6 additions and 5 deletions

CurlStorage.cxx src/storage/plugins/CurlStorage.cxx +6 -5

No files found.
--- a/src/storage/plugins/CurlStorage.cxx
+++ b/src/storage/plugins/CurlStorage.cxx
@@ -262,18 +262,19 @@ public:
 		request.SetOption(CURLOPT_MAXREDIRS, 1L);

 		request_headers.Append(StringFormat<40>("depth: %u", depth));
+		request_headers.Append("content-type: text/xml");

 		request.SetOption(CURLOPT_HTTPHEADER, request_headers.Get());

 		request.SetOption(CURLOPT_POSTFIELDS,
 				  "<?xml version=\"1.0\"?>\n"
 				  "<a:propfind xmlns:a=\"DAV:\">"
-				  "<a:prop><a:resourcetype/></a:prop>"
-				  "<a:prop><a:getcontenttype/></a:prop>"
-				  "<a:prop><a:getcontentlength/></a:prop>"
+				  "<a:prop>"
+				  "<a:resourcetype/>"
+				  "<a:getcontenttype/>"
+				  "<a:getcontentlength/>"
+				  "</a:prop>"
 				  "</a:propfind>");
-
-		// TODO: send request body
 	}

 	using BlockingHttpRequest::GetEasy;