Skip to content

Commit 46f4df4

Browse files
committed
fix: Improve logging for JSON serialization methods
1 parent b3863fa commit 46f4df4

1 file changed

Lines changed: 7 additions & 2 deletions

File tree

ReflectorNet/src/Extension/ExtensionsJson.cs

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,10 @@ public static class ExtensionsJson
1616
{
1717
public static JsonElement ToJsonElement(this object data, Reflector? reflector, JsonSerializerOptions? options = null, ILogger? logger = null)
1818
{
19-
logger?.LogTrace("Converting object of type {Type} to JsonElement.", data?.GetType().GetTypeId() ?? "null");
19+
if (logger?.IsEnabled(LogLevel.Trace) == true)
20+
logger.LogTrace("Converting object of type {Type} to JsonElement.",
21+
data?.GetType().GetTypeId().ValueOrNull());
22+
2023
return JsonSerializer.SerializeToElement(data, options ?? reflector?.JsonSerializerOptions);
2124
}
2225

@@ -51,7 +54,9 @@ public static string ToJson(this object? value, string defaultValue, Reflector?
5154
if (value is Utils.JsonSerializer)
5255
throw new ArgumentException("Cannot serialize JsonSerializer instance.", nameof(value));
5356

54-
logger?.LogTrace("Serializing object of type {Type} to JSON string.", value.GetType().GetTypeId());
57+
if (logger?.IsEnabled(LogLevel.Trace) == true)
58+
logger.LogTrace("Serializing object of type {Type} to JSON string.",
59+
value.GetType().GetTypeId().ValueOrNull());
5560

5661
return JsonSerializer.Serialize(
5762
value: value,

0 commit comments

Comments
 (0)