From c33b4e252f8bf37bc716adc7789926185c3e384c Mon Sep 17 00:00:00 2001 From: Petr Mrázek Date: Sun, 21 Jan 2018 03:49:54 +0100 Subject: NOISSUE fix bad redirect URLs provided by the curse CDN MultiMC now parses the HTTP Location header in a (more) tolerant mode. --- api/logic/minecraft/flame/FileResolvingTask.cpp | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) (limited to 'api/logic/minecraft/flame/FileResolvingTask.cpp') diff --git a/api/logic/minecraft/flame/FileResolvingTask.cpp b/api/logic/minecraft/flame/FileResolvingTask.cpp index af8ef7aa..980b8c3a 100644 --- a/api/logic/minecraft/flame/FileResolvingTask.cpp +++ b/api/logic/minecraft/flame/FileResolvingTask.cpp @@ -48,7 +48,13 @@ void Flame::FileResolvingTask::netJobFinished() continue; } out.fileName = Json::requireString(obj, "FileNameOnDisk"); - out.url = Json::requireString(obj, "DownloadURL"); + auto urlString = Json::requireString(obj, "DownloadURL"); + urlString.replace(' ', "%20"); + out.url = QUrl(urlString, QUrl::StrictMode); + if(!out.url.isValid()) + { + throw "Perkele!"; + } // This is a piece of a Flame project JSON pulled out into the file metadata (here) for convenience // It is also optional QJsonObject projObj = Json::ensureObject(obj, "_Project", {}); -- cgit