Support IntPtr and UIntPtr serialization (#1746)

Author: SimonCropp

CHANGELOG.md

@@ -7,6 +7,7 @@
 - Expose EnumerateChainedExceptions ([#1733](https://github.com/getsentry/sentry-dotnet/pull/1733))
 - Android Scope Sync ([#1737](https://github.com/getsentry/sentry-dotnet/pull/1737))
 - Enable logging in MAUI ([#1738](https://github.com/getsentry/sentry-dotnet/pull/1738))
+- Support IntPtr and UIntPtr serialization ([#1746](https://github.com/getsentry/sentry-dotnet/pull/1746))
 - Catch permission exceptions on Android ([#1750](https://github.com/getsentry/sentry-dotnet/pull/1750))
 
 ## Sentry.Maui 3.18.0-preview.1

src/Sentry/Internal/Extensions/JsonExtensions.cs

@@ -11,7 +11,17 @@ internal static class JsonExtensions
     {
         // The Json options with a preset of rules that will remove dangerous and problematic
         // data from the serialized object.
-        private static JsonSerializerOptions serializerOption = new() { Converters = { new SentryJsonConverter() } };
+        private static JsonSerializerOptions serializerOption = new()
+        {
+            Converters =
+            {
+                new SentryJsonConverter(),
+                new IntPtrJsonConverter(),
+                new IntPtrNullableJsonConverter(),
+                new UIntPtrJsonConverter(),
+                new UIntPtrNullableJsonConverter()
+            }
+        };
 
         public static void Deconstruct(this JsonProperty jsonProperty, out string name, out JsonElement value)
         {

src/Sentry/Internal/JsonConverters/IntPtrJsonConverter.cs

@@ -0,0 +1,19 @@
+using System;
+using System.Text.Json;
+using System.Text.Json.Serialization;
+
+namespace Sentry.Internal
+{
+    internal class IntPtrJsonConverter : JsonConverter<IntPtr>
+    {
+        public override IntPtr Read(ref Utf8JsonReader reader, Type typeToConvert, JsonSerializerOptions options)
+        {
+            return new IntPtr(reader.GetInt64());
+        }
+
+        public override void Write(Utf8JsonWriter writer, IntPtr value, JsonSerializerOptions options)
+        {
+            writer.WriteNumberValue(value.ToInt64());
+        }
+    }
+}

src/Sentry/Internal/JsonConverters/IntPtrNullableJsonConverter.cs

@@ -0,0 +1,31 @@
+using System;
+using System.Text.Json;
+using System.Text.Json.Serialization;
+
+namespace Sentry.Internal
+{
+    internal class IntPtrNullableJsonConverter : JsonConverter<IntPtr?>
+    {
+        public override IntPtr? Read(ref Utf8JsonReader reader, Type typeToConvert, JsonSerializerOptions options)
+        {
+            if (reader.TokenType == JsonTokenType.Null)
+            {
+                return null;
+            }
+
+            return new IntPtr(reader.GetInt64());
+        }
+
+        public override void Write(Utf8JsonWriter writer, IntPtr? value, JsonSerializerOptions options)
+        {
+            if (value == null)
+            {
+                writer.WriteNullValue();
+            }
+            else
+            {
+                writer.WriteNumberValue(value.Value.ToInt64());
+            }
+        }
+    }
+}

src/Sentry/Internal/SentryJsonConverter.cs renamed to src/Sentry/Internal/JsonConverters/SentryJsonConverter.cs

@@ -0,0 +1,19 @@
+using System;
+using System.Text.Json;
+using System.Text.Json.Serialization;
+
+namespace Sentry.Internal
+{
+    internal class UIntPtrJsonConverter : JsonConverter<UIntPtr>
+    {
+        public override UIntPtr Read(ref Utf8JsonReader reader, Type typeToConvert, JsonSerializerOptions options)
+        {
+            return new UIntPtr(reader.GetUInt64());
+        }
+
+        public override void Write(Utf8JsonWriter writer, UIntPtr value, JsonSerializerOptions options)
+        {
+            writer.WriteNumberValue(value.ToUInt64());
+        }
+    }
+}

src/Sentry/Internal/JsonConverters/UIntPtrJsonConverter.cs

@@ -0,0 +1,31 @@
+using System;
+using System.Text.Json;
+using System.Text.Json.Serialization;
+
+namespace Sentry.Internal
+{
+    internal class UIntPtrNullableJsonConverter : JsonConverter<UIntPtr?>
+    {
+        public override UIntPtr? Read(ref Utf8JsonReader reader, Type typeToConvert, JsonSerializerOptions options)
+        {
+            if (reader.TokenType == JsonTokenType.Null)
+            {
+                return null;
+            }
+
+            return new UIntPtr(reader.GetUInt64());
+        }
+
+        public override void Write(Utf8JsonWriter writer, UIntPtr? value, JsonSerializerOptions options)
+        {
+            if (value == null)
+            {
+                writer.WriteNullValue();
+            }
+            else
+            {
+                writer.WriteNumberValue(value.Value.ToUInt64());
+            }
+        }
+    }
+}

src/Sentry/Internal/JsonConverters/UIntPtrNullableJsonConverter.cs

@@ -0,0 +1 @@
+5

test/Sentry.Tests/SerializationTests.Serialization_name=int.verified.txt

@@ -0,0 +1 @@
+{"Value":3}

test/Sentry.Tests/SerializationTests.Serialization_name=nested IntPtr.verified.txt

@@ -0,0 +1 @@
+{"Value":3}