mirrors/Ryujinx
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

prepo: Resolve JSON parsing issues in prepo report handling

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-24 23:33:45 +11:00
parent e590262531
commit 8bc4c993f9
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();
}
}
}