Skip to content

Commit 6987e46

Browse files
committed
Add try/catch for failure in SerializeDebugData
1 parent 487b56e commit 6987e46

File tree

1 file changed

+17
-9
lines changed

1 file changed

+17
-9
lines changed

Src/StackifyLib/Utils/HelperFunctions.cs

Lines changed: 17 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -205,20 +205,28 @@ public static string SerializeDebugData(object logObject, bool serializeSimpleTy
205205

206206
if (jObject != null)
207207
{
208-
209-
jObject = GetPrunedObject(jObject, Config.LoggingJsonMaxFields);
210-
211-
string json;
212-
using (var writer = new StringWriter())
208+
try
213209
{
214-
using (var jsonWriter = new MaxDepthJsonTextWriter(writer, Config.LoggingJsonMaxDepth))
210+
jObject = GetPrunedObject(jObject, Config.LoggingJsonMaxFields);
211+
212+
string json;
213+
using (var writer = new StringWriter())
215214
{
216-
JsonSerializer.CreateDefault().Serialize(jsonWriter, jObject);
217-
json = writer.ToString();
215+
using (var jsonWriter = new MaxDepthJsonTextWriter(writer, Config.LoggingJsonMaxDepth))
216+
{
217+
JsonSerializer.CreateDefault().Serialize(jsonWriter, jObject);
218+
json = writer.ToString();
219+
}
218220
}
221+
222+
return json;
219223
}
224+
catch (JsonSerializationException ex)
225+
{
226+
Utils.StackifyAPILogger.Log(ex.ToString(), true);
220227

221-
return json;
228+
return null;
229+
}
222230
}
223231

224232
return null;

0 commit comments

Comments
 (0)