Url escape roomid in /messages and relax parsing of response

lib/http/client.cpp

@@ -751,8 +751,8 @@ Client::messages(const MessagesOpts &opts, Callback<mtx::responses::Messages> ca
         if (!opts.filter.empty())
                 params.emplace("filter", opts.filter);
 
-        const auto api_path =
-          "/client/r0/rooms/" + opts.room_id + "/messages?" + client::utils::query_params(params);
+        const auto api_path = "/client/r0/rooms/" + mtx::client::utils::url_encode(opts.room_id) +
+                              "/messages?" + client::utils::query_params(params);
 
         get<mtx::responses::Messages>(
           api_path, [callback](const mtx::responses::Messages &res, HeaderFields, RequestErr err) {

lib/structs/responses/messages.cpp

@@ -9,10 +9,11 @@ namespace responses {
 void
 from_json(const json &obj, Messages &messages)
 {
-        messages.start = obj.at("start").get<std::string>();
-        messages.end   = obj.at("end").get<std::string>();
+        messages.start = obj.value("start", "");
+        messages.end   = obj.value("end", "");
 
-        utils::parse_timeline_events(obj.at("chunk"), messages.chunk);
+        if (obj.contains("chunk"))
+                utils::parse_timeline_events(obj.at("chunk"), messages.chunk);
 }
 }
 }