From 78a089cd70aacfc65fafaf06c3099c9ad5a3cc68 Mon Sep 17 00:00:00 2001 From: Denis Andrasec Date: Mon, 6 May 2024 14:26:29 +0200 Subject: [PATCH 1/2] read type from platform exception --- ...id_platform_exception_event_processor.dart | 7 +- ...atform_exception_event_processor_test.dart | 67 +++++++++++++++++++ 2 files changed, 72 insertions(+), 2 deletions(-) diff --git a/flutter/lib/src/event_processor/android_platform_exception_event_processor.dart b/flutter/lib/src/event_processor/android_platform_exception_event_processor.dart index 6570dedd81..bffc6707b7 100644 --- a/flutter/lib/src/event_processor/android_platform_exception_event_processor.dart +++ b/flutter/lib/src/event_processor/android_platform_exception_event_processor.dart @@ -153,11 +153,14 @@ extension on JvmException { String? module; final typeParts = type?.split('.'); if (typeParts != null) { - if (typeParts.length > 1) { + if (typeParts.isNotEmpty) { exceptionType = typeParts.last; } typeParts.remove(typeParts.last); - module = typeParts.join('.'); + + if (typeParts.isNotEmpty) { + module = typeParts.join('.'); + } } final stackFrames = stackTrace.asMap().entries.map((entry) { return entry.value.toSentryStackFrame(entry.key, nativePackageName); diff --git a/flutter/test/android_platform_exception_event_processor_test.dart b/flutter/test/android_platform_exception_event_processor_test.dart index 5e916f06a3..060ada138c 100644 --- a/flutter/test/android_platform_exception_event_processor_test.dart +++ b/flutter/test/android_platform_exception_event_processor_test.dart @@ -51,6 +51,32 @@ void main() { expect(platformException_2.stackTrace!.frames.length, 18); }); + test('platform exception is correctly parsed', () async { + final platformExceptionEvent = await fixture.processor + .apply(fixture.eventWithFailingPlatformStackTrace, Hint()); + + final exceptions = platformExceptionEvent!.exceptions!; + expect(exceptions.length, 3); + + final platformException = exceptions[1]; + + expect(platformException.type, 'PlatformException'); + expect( + platformException.value, + "PlatformException(getNotificationChannelsError, Unable to find resource ID #0x7f14000d, android.content.res.Resources\$NotFoundException: Unable to find resource ID #0x7f14000d", + ); + expect(platformException.stackTrace!.frames.length, 20); + + final platformException_2 = exceptions[2]; + + expect(platformException_2.type, 'PlatformException'); + expect( + platformException_2.value, + "PlatformException(getNotificationChannelsError, Unable to find resource ID #0x7f14000d, android.content.res.Resources\$NotFoundException: Unable to find resource ID #0x7f14000d", + ); + expect(platformException_2.stackTrace!.frames.length, 20); + }); + test( 'Dart thread is current and not crashed if Android exception is present', () async { @@ -142,6 +168,16 @@ class Fixture { threads: [dartThread], ); + late SentryException withFailingPlatformStackTrace = options.exceptionFactory + .getSentryException(failingPlatformException) + .copyWith(threadId: 1); + + late SentryEvent eventWithFailingPlatformStackTrace = SentryEvent( + exceptions: [withFailingPlatformStackTrace], + throwable: failingPlatformException, + threads: [dartThread], + ); + late SentryThread dartThread = SentryThread( crashed: true, current: true, @@ -169,6 +205,14 @@ final emptyPlatformException = PlatformException( stacktrace: null, ); +final failingPlatformException = PlatformException( + code: 'error', + details: + "PlatformException: PlatformException(getNotificationChannelsError, Unable to find resource ID #0x7f14000d, android.content.res.Resources\$NotFoundException: Unable to find resource ID #0x7f14000d", + message: _failingStackTrace, + stacktrace: _failingStackTrace, +); + const _jvmStackTrace = """java.lang.IllegalArgumentException: Unsupported value: '[Ljava.lang.StackTraceElement;@ba6feed' of type 'class [Ljava.lang.StackTraceElement;' at io.flutter.plugin.common.StandardMessageCodec.writeValue(StandardMessageCodec.java:292) @@ -189,3 +233,26 @@ const _jvmStackTrace = at java.lang.reflect.Method.invoke(Native Method) at com.android.internal.os.RuntimeInit\$MethodAndArgsCaller.run(RuntimeInit.java:556) at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1037)"""; + +const _failingStackTrace = + """PlatformException: PlatformException(getNotificationChannelsError, Unable to find resource ID #0x7f14000d, android.content.res.Resources\$NotFoundException: Unable to find resource ID #0x7f14000d + at android.content.res.ResourcesImpl.getResourceEntryName(ResourcesImpl.java:493) + at android.content.res.Resources.getResourceEntryName(Resources.java:2441) + at com.dexterous.flutterlocalnotifications.FlutterLocalNotificationsPlugin.getMappedNotificationChannel(FlutterLocalNotificationsPlugin.java:170) + at com.dexterous.flutterlocalnotifications.FlutterLocalNotificationsPlugin.getNotificationChannels(FlutterLocalNotificationsPlugin.java:32) + at com.dexterous.flutterlocalnotifications.FlutterLocalNotificationsPlugin.onMethodCall(FlutterLocalNotificationsPlugin.java:399) + at be.j\$a.a(MethodChannel.java:18) + at pd.c.l(DartMessenger.java:19) + at pd.c.m(DartMessenger.java:42) + at pd.c.h(Unknown Source:0) + at pd.b.run(Unknown Source:12) + at android.os.Handler.handleCallback(Handler.java:966) + at android.os.Handler.dispatchMessage(Handler.java:110) + at android.os.Looper.loopOnce(Looper.java:205) + at android.os.Looper.loop(Looper.java:293) + at android.app.ActivityThread.loopProcess(ActivityThread.java:9832) + at android.app.ActivityThread.main(ActivityThread.java:9821) + at java.lang.reflect.Method.invoke(Native Method) + at com.android.internal.os.RuntimeInit\$MethodAndArgsCaller.run(RuntimeInit.java:586) + at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1201) +, null)"""; From bee329d1b341210c56485f332ef9bdcdac5b3df7 Mon Sep 17 00:00:00 2001 From: Denis Andrasec Date: Mon, 6 May 2024 14:34:21 +0200 Subject: [PATCH 2/2] add changelog entry --- CHANGELOG.md | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/CHANGELOG.md b/CHANGELOG.md index c545ea3e39..9b4f7e3f6e 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -6,6 +6,10 @@ - Adds app start spans to first transaction ([#2009](https://github.com/getsentry/sentry-dart/pull/2009)) +### Fixes + +- Fix `PlatformException` title parsing ([#2033](https://github.com/getsentry/sentry-dart/pull/2033)) + ## 8.1.0 ### Features