Skip to content

Commit 4f00956

Browse files
jschick04NikTilton
authored andcommitted
Added failure path if Deserialize is actually null
1 parent 401b274 commit 4f00956

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

src/EventLogExpert.Eventing/EventProviderDatabase/CompressedJsonValueConverter.cs

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,11 +33,13 @@ public static T ConvertFromCompressedJson(byte[] value)
3333
using GZipStream gZipStream = new(memoryStream, CompressionMode.Decompress);
3434
using StreamReader streamReader = new(gZipStream);
3535

36-
return JsonSerializer.Deserialize<T>(streamReader.ReadToEnd())!;
36+
return JsonSerializer.Deserialize<T>(streamReader.ReadToEnd())
37+
?? throw new JsonException($"Failed to deserialize compressed JSON to type {typeof(T).Name}. The deserialized value was null.");
3738
}
3839

3940
private static T ConvertFromJson(string value)
4041
{
41-
return JsonSerializer.Deserialize<T>(value)!;
42+
return JsonSerializer.Deserialize<T>(value)
43+
?? throw new JsonException($"Failed to deserialize JSON to type {typeof(T).Name}. The deserialized value was null.");
4244
}
4345
}

0 commit comments

Comments
 (0)