From daecb1193de369390ee0efd0a8dc4c21de2ed9ba Mon Sep 17 00:00:00 2001 From: jduncanator <1518948+jduncanator@users.noreply.github.com> Date: Wed, 25 Mar 2020 09:26:37 +1100 Subject: [PATCH] prepo: Resolve JSON parsing issues in prepo report handling (#1022) It seems MsgPack.Cli incorrectly converts some unicode control characters directly to JSON literal Unicode strings (eg. '\1'). As these are invalid, pretty printing the JSON report fails. This removes pretty printing, pending a proper implementation, and tidies up MsgPack deserialization. --- .../HOS/Services/Prepo/IPrepoService.cs | 31 ++++++------------- 1 file changed, 9 insertions(+), 22 deletions(-) diff --git a/Ryujinx.HLE/HOS/Services/Prepo/IPrepoService.cs b/Ryujinx.HLE/HOS/Services/Prepo/IPrepoService.cs index 67344061..c2857447 100644 --- a/Ryujinx.HLE/HOS/Services/Prepo/IPrepoService.cs +++ b/Ryujinx.HLE/HOS/Services/Prepo/IPrepoService.cs @@ -1,3 +1,4 @@ +using MsgPack; using MsgPack.Serialization; using Ryujinx.Common; using Ryujinx.Common.Logging; @@ -83,35 +84,21 @@ namespace Ryujinx.HLE.HOS.Services.Prepo private string ReadReportBuffer(byte[] buffer, string room, UserId userId) { - StringBuilder sb = new StringBuilder(); + StringBuilder builder = new StringBuilder(); + MessagePackObject deserializedReport = MessagePackSerializer.UnpackMessagePackObject(buffer); - sb.AppendLine(); - sb.AppendLine("PlayReport log:"); + builder.AppendLine(); + builder.AppendLine("PlayReport log:"); if (!userId.IsNull) { - sb.AppendLine($" UserId: {userId.ToString()}"); + builder.AppendLine($" UserId: {userId.ToString()}"); } - sb.AppendLine($" Room: {room}"); + builder.AppendLine($" Room: {room}"); + builder.AppendLine($" Report: {deserializedReport}"); - var deserializedReport = Deserialize(buffer); - string jsonReport = Encoding.ASCII.GetString(JsonSerializer.PrettyPrintByteArray(Encoding.UTF8.GetBytes(deserializedReport.ToString()))); - - sb.AppendLine($" Report:"); - sb.AppendLine(jsonReport); - - return sb.ToString(); - } - - private static T Deserialize(byte[] bytes) - { - MessagePackSerializer serializer = MessagePackSerializer.Get(); - - using (MemoryStream byteStream = new MemoryStream(bytes)) - { - return (T)serializer.Unpack(byteStream); - } + return builder.ToString(); } } } \ No newline at end of file