invertase
🐛 [@react-native-firebase/crashlytics] Add support for enabling gwpAsanMode in android config plugin
The crashlytics dashboard is alerting me to enable GWP-ASan:
Your app has NDK crashes that can lead to security vulnerabilities. Enable GWP-ASan to see additional debugging data in Crashlytics Learn more
The firebase BoM v31.3.0+ supports this feature, and react-antive-firebase uses a newer version: https://github.com/invertase/react-native-firebase/blob/c4959a1dcba577a8f6f0f87c16e5d247169cd383/packages/app/package.json#L86
https://github.com/invertase/react-native-firebase/blob/c4959a1dcba577a8f6f0f87c16e5d247169cd383/packages/crashlytics/android/build.gradle#L34
I'm considering writing a config plugin to do just that, and was considering contributing it to react-native-firebase.
My instinct is to put the config in the expo config plugin's parameters, and not the firebase.json file, as the feature I'm suggesting is a config plugin only.
Thoughts?
Hi there, seeing it is supported in the native Android sdk, we should be able to support it here, I am happy to give this a crack myself.
Fascinating, didn't know this existed. @MichaelVerdon appears you may be able to use firebase.json in combo with manifestPlaceholders style - used in analytics/android/build.gradle + analytics/android/src/main/AndroidManifest.xml
May work to interpolate the value in there? I think the default has to be off since it impacts memory usage and may cause app termination, then as a basic implementation just allowing the mode to be set (if library users want to use it...) would be great
