Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🐛[Bug Report]: AuthPrompt is the source of many crashes #241

Open
kcw-grunt opened this issue Oct 6, 2024 · 3 comments · May be fixed by #256
Open

🐛[Bug Report]: AuthPrompt is the source of many crashes #241

kcw-grunt opened this issue Oct 6, 2024 · 3 comments · May be fixed by #256
Assignees
Labels
🦟 critical bug Drop everything. this bug needs immediate attention
Milestone

Comments

@kcw-grunt
Copy link
Member

kcw-grunt commented Oct 6, 2024

Device model

A wide number of devices are experiencing this crash

App Crashes Distribution
Screenshot 2024-10-06 at 8 17 37 PM

OS version

Other

Which version of Litewallet are you running?

Other

Describe what happened.

  1. User launches the app
  2. The method causes crashes
  3. In this case the app crashes

Add any notes you would like to add.

Firebase notes:

https://console.firebase.google.com/project/litewallet-beta/crashlytics/app/android:com.loafwallet/issues/e62bd3b2c450bdc67fa703c10e2ce5db?time=1712994462715:1715586462715&sessionEventKey=664116C701AD00014458EB7DA4CD049D_1946818051493448057

Code snippet found

Please see the method where the code is causing the problem. (In AuthManager.java) on lines in 218, 231

public void authPrompt(final Context context, String title, String message, boolean forcePin, boolean forceFingerprint, BRAuthCompletion completion)
@kcw-grunt kcw-grunt added the 🦟 critical bug Drop everything. this bug needs immediate attention label Oct 6, 2024
@josikie
Copy link
Member

josikie commented Oct 6, 2024

Ok, noted!

@kcw-grunt
Copy link
Member Author

we are working on this!

@kcw-grunt kcw-grunt assigned kcw-grunt and josikie and unassigned josikie Nov 5, 2024
@kcw-grunt
Copy link
Member Author

cc: @andhikayuana

We will need your review

@kcw-grunt kcw-grunt linked a pull request Nov 5, 2024 that will close this issue
4 tasks
@kcw-grunt kcw-grunt linked a pull request Nov 5, 2024 that will close this issue
4 tasks
@kcw-grunt kcw-grunt added this to the v2.12.X milestone Nov 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🦟 critical bug Drop everything. this bug needs immediate attention
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants