fix: vendorId is a String in sentry-java (#1813)

getsentry · Sep 26, 2024 · b534643 · b534643
1 parent b28a2d9
commit b534643
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,6 +2,10 @@
 
 ## Unreleased
 
+### Fixes
+
+- The SDK no longer passes a mangled `gpu.vendorId` to the Android native layer ([#1813](https://github.com/getsentry/sentry-unity/pull/1813))
+
 ### Features
 
 - Contexts, such as `device` and `gpu` that the SDK retrieves during the game's startup is now available even earlier and irrespective whether an error was captured on the main or on a background thread ([#1802](https://github.com/getsentry/sentry-unity/pull/1802))

diff --git a/src/Sentry.Unity.Android/SentryJava.cs b/src/Sentry.Unity.Android/SentryJava.cs
@@ -103,11 +103,7 @@ public void WriteScope(
             using var gpu = new AndroidJavaObject("io.sentry.protocol.Gpu");
             gpu.SetIfNotNull("name", GpuName);
             gpu.SetIfNotNull("id", GpuId);
-            if (GpuVendorId is not null && int.TryParse(GpuVendorId, out var intVendorId) && intVendorId != 0)
-            {
-                using var integer = new AndroidJavaObject("java.lang.Integer", intVendorId);
-                gpu.Set("vendorId", integer);
-            }
+            gpu.SetIfNotNull("vendorId", GpuVendorId);
             gpu.SetIfNotNull("vendorName", GpuVendorName);
             gpu.SetIfNotNull("memorySize", GpuMemorySize);
             gpu.SetIfNotNull("apiType", GpuApiType);