Improve/add info for symbolication and deobfuscated crash reports

mapbox / mapbox-android-demo

Google Play demo app for the Mapbox Maps SDK for Android

https://play.google.com/store/apps/details?id=com.mapbox.mapboxandroiddemo

BSD 2-Clause "Simplified" License

720 stars 493 forks source link

Improve/add info for symbolication and deobfuscated crash reports #1122

Open langsmith opened 5 years ago

langsmith commented 5 years ago

Making this ticket to track @zugaldia's suggestion that we improve our ability to look into this app's crash reports. This might include having to update some sort of metadata in our Firebase account.

https://firebase.google.com/docs/crashlytics/get-deobfuscated-reports?platform=android

tobrun commented 5 years ago

I would be highly interested in integrating https://firebase.google.com/docs/crashlytics/ndk-reports. This however will result in having to build the Mapbox Maps SDK for Android from source (eg. as a submodule vs a binary).

cc @LukasPaczos

zugaldia commented 5 years ago

This however will result in having to build the Mapbox Maps SDK for Android from source

Could we do this for all builds of the Maps SDK instead so that we don't need to build from source? Or, alternatively, could we have a specific a crashlytics build with this set up enabled so that us and external devs could take advantage of?

LukasPaczos commented 5 years ago

All of the pieces in the dependency tree of crashlytics/ndk-reports seem to weigh over 900kB (at least in the raw aar form) so I'd lean towards a specific crashlytics build to help debug the issues downstream.