discuss: Android catches all plugin exceptions and logs message while iOS allows them to crash the app. Should they be made to work the same across platforms?

In trying to create a plugin for Firebase Crashlytics, it seems that @capacitor/ios allows exceptions within a plugin method to bubble all the way up and crash the app and @capacitor/android catches all exceptions and logs a simple message. Android code is here and here.

To me, the Android implementation seems like it could hide problems within the plugin, never to be seen or reported by tools like Crashlytics. While good intentioned, this seems like a bad idea overall.

Would the Capacitor team welcome PRs that remove the exception catching logic from the android platform?

[josh-m-sharpe]

I think that approach would work too. Either way, I suspect it makes sense to update all platforms to work the same.

It would be good to hear from the internal team on this though.

[ionitron-bot[bot]]

Thanks for the issue! This issue is being locked to prevent comments that are not relevant to the original issue. If this is still an issue with the latest version of Capacitor, please create a new issue and ensure the template is fully filled out.