PlatformException.message should be treated like PlatformException.stacktrace

[ueman]

Problem Statement

In theory, only PlatformException.stacktrace should contain a stacktrace. However, in practice I see a lot of plugins misusing the PlatformException.message field for stacktraces.

Solution Brainstorm

Therefore, it would be nice if the PlatformException.message field could be treated like the PlatformException.stacktrace field.

Are you willing to submit a PR?

Yes

[buenaflor]

Do you have an example of how it's misused? Not sure if I understand correctly if message doesn't contain a stacktrace

[ueman]

Sure.

Here's an example where a native stacktrace is passed to Flutter in the PlatformException.message field:

As you can see, it's not really well formatted and Sentry could do better.

Here's an example of how it looks like when the native stacktrace is passed via the PlatformException.stacktrace field and how I want it to look like for the example above

It should be a straight forward change in the following file https://github.com/getsentry/sentry-dart/blob/955541a50762f04c301c83eb25aef0077742c29a/flutter/lib/src/event_processor/android_platform_exception_event_processor.dart

Basically the new logic should be in case PlatformException.stacktrace is null, try to parse PlatformException.message instead.

[buenaflor]

I see, makes sense if it is parsed as a fallback 👍