https://bugs.documentfoundation.org/show_bug.cgi?id=146703

Julien Nabet <serval2...@yahoo.fr> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
     Ever confirmed|0                           |1
             Status|UNCONFIRMED                 |NEW
                 CC|                            |michael.st...@allotropia.de
                   |                            |, serval2...@yahoo.fr

--- Comment #2 from Julien Nabet <serval2...@yahoo.fr> ---
On pc Debian x86-64 with master sources updated today, I could reproduce this.

I got same console logs as you.

Some gdb debug show:
http_dav_ucp::CurlUri::CurlUri (this=0x7faeab39af20,
rURI=u"https://extensions.libreoffice.org/api/v0/Color Palette.json")

since LO goes into if (uc != CURLUE_OK), here's the value of uc:
CURLUE_MALFORMED_INPUT

Michael: I tried this patch and it worked:
diff --git a/ucb/source/ucp/webdav-curl/CurlUri.cxx
b/ucb/source/ucp/webdav-curl/CurlUri.cxx
index 035c12a132ef..1e96d7b140f3 100644
--- a/ucb/source/ucp/webdav-curl/CurlUri.cxx
+++ b/ucb/source/ucp/webdav-curl/CurlUri.cxx
@@ -116,7 +116,7 @@ CurlUri::CurlUri(::std::u16string_view const rURI)

     // use curl to parse the URI, to get a consistent interpretation
     OString const utf8URI(OUStringToOString(rURI, RTL_TEXTENCODING_UTF8));
-    auto uc = curl_url_set(m_pUrl.get(), CURLUPART_URL, utf8URI.getStr(), 0);
+    auto uc = curl_url_set(m_pUrl.get(), CURLUPART_URL, utf8URI.getStr(),
CURLU_ALLOW_SPACE);
     if (uc != CURLUE_OK)
     {
         SAL_WARN("ucb.ucp.webdav.curl", "curl_url_set failed: " << uc);


Should we use CURLU_ALLOW_SPACE or should the url be encoded in a specific way?

-- 
You are receiving this mail because:
You are the assignee for the bug.

Reply via email to