XITRIX/MeloNX
1
0
Fork 0
forked from MeloNX/MeloNX
Code Pull Requests Activity

prepo: Resolve JSON parsing issues in prepo report handling (#1022)

Browse Source 
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.
This commit is contained in:
jduncanator 2020-03-25 09:26:37 +11:00 committed by GitHub
parent 8be3c73ade
commit 76f12ceb05
1 changed files with 9 additions and 22 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

31
Ryujinx.HLE/HOS/Services/Prepo/IPrepoService.cs
View File

@ -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<dynamic>(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<T>(byte[] bytes)
{
MessagePackSerializer serializer = MessagePackSerializer.Get<T>();
using (MemoryStream byteStream = new MemoryStream(bytes))
{
return (T)serializer.Unpack(byteStream);
}
return builder.ToString();
}
}
}