search

celo-org / celo-monorepo

Official repository for core projects comprising the Celo platform
https://celo.org
Apache License 2.0
700 stars 369 forks source link

[Android] App get crashed when user taps on “Send an issue report” option from the “Setting” screen. #1437

Closed Lss-Ankit closed 4 years ago

Lss-Ankit commented 5 years ago

Frequency: 100% App version: zero-sync build: Staging build v1.5.0 (101548577) Repro on: Samsung Galaxy Grand Prime (5.1.1), Samsung Galaxy s5 (6.0.1) Testing Account: 680 208 0265/Backup Kay

Repro Steps: 1) Launch the app. 2) Tap on the “Cog” icon. 3) Tap on send an issue report. 4) Observe.

Impact: User is blocked to send an issue log. Current Behavior: App gets crashed. Expected Behavior: User should be able to create an issue log Attachment: 1) Android_AppGetCrashed.mp4 (Bug observe at 6 sec) 2) Crashlog.txt

jmrossy commented 5 years ago

It seems he hit an OOM error. The logs can get big. I'll see if we can patch this in an easy way

jmrossy commented 5 years ago

Related: https://github.com/itinance/react-native-fs/issues/598

jmrossy commented 5 years ago

We may be able to switch to using https://github.com/joltup/rn-fetch-blob instead to stream the file gradually.

Lss-Ankit commented 4 years ago

Verified with latest Android build v1.6.1 and found that issue is fixed as now user is able to send the report using "Send an issue report" option from setting.

I have also verify the issue on latest iOS Test flight build v1.6.1 (23) and found the following

Attachment: IOS_SendReportIssue.mp4

jmrossy commented 4 years ago

@Lss-Ankit thank you for verifying but actually this is not yet fixed. However the crash will only occur on low-end phones when the logs get large.

As for iOS, we have fixed that and the fix will go out with the next app update.

Lss-Ankit commented 4 years ago

@jmrossy Thanks for the update. I have verified this issue in Android 5 & 6 os devices and issue is fixed now in play store build : 1.6.1 (104794404).

Verified on Account : 3155641283/Backupkey (quiz egg lumber aisle juice impose control lens office caught turn skull rude planet merge hub cross purchase gown castle dinner steak license depart)

Verified on Devices : Samsung Galaxy S5 (6.0.1) & Samsung Galaxy Grand Prime (5.0.1)

Attachment : Android 5 & 6_App does not get crashed