[Libreoffice-commits] online.git: Branch 'distro/collabora/collabora-online-4-0' - wsd/LOOLWSD.cpp
Szymon KÅos (via logerrit)
logerrit at kemper.freedesktop.org
Wed Sep 16 18:07:52 UTC 2020
wsd/LOOLWSD.cpp | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
New commits:
commit 9351ec6429e7985f9fc5aeb80e21316b37291c71
Author: Szymon Kłos <szymon.klos at collabora.com>
AuthorDate: Wed Sep 16 16:41:09 2020 +0200
Commit: Andras Timar <andras.timar at collabora.com>
CommitDate: Wed Sep 16 20:07:47 2020 +0200
Decode file name when trying to download file
Simplified download process (ab162b6f9580315700a01c3bc10becd510a2ead4)
introduced a bug. Trying to download file with eg. space in the name
become impossible. This patch decodes the string to fix the issue.
Change-Id: I8e5d98010cf098b889eddcce4ba1ce65367b15d5
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/102874
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice at gmail.com>
Reviewed-by: Szymon Kłos <szymon.klos at collabora.com>
(cherry picked from commit 87aad0a8bbf86502658ffcf3d0ee053af922d474)
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/102763
Reviewed-by: Andras Timar <andras.timar at collabora.com>
diff --git a/wsd/LOOLWSD.cpp b/wsd/LOOLWSD.cpp
index 4f8e108fa..4b78d2096 100644
--- a/wsd/LOOLWSD.cpp
+++ b/wsd/LOOLWSD.cpp
@@ -2542,7 +2542,10 @@ private:
bool foundDownloadId = !url.empty();
- const Path filePath(LOOLWSD::ChildRoot + jailId + JAILED_DOCUMENT_ROOT + url);
+ std::string decoded;
+ Poco::URI::decode(url, decoded);
+
+ const Path filePath(LOOLWSD::ChildRoot + jailId + JAILED_DOCUMENT_ROOT + decoded);
const std::string filePathAnonym = LOOLWSD::anonymizeUrl(filePath.toString());
if (foundDownloadId && filePath.isAbsolute() && File(filePath).exists())
More information about the Libreoffice-commits
mailing list