-
Notifications
You must be signed in to change notification settings - Fork 578
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
Firebase Auth crash when R8 full mode obfuscation enabled #2124
Comments
I couldn't figure out how to label this issue, so I've labeled it for a human to triage. Hang tight. |
Hey there! I've filed b/172259754 to track this internally :) |
Ok, it appears that this only happens when you use To be noted: I think the root cause is a use of reflection at the network layer of the SDK. I don't yet know how to protect classes when using enableR8, so it's gonna take me a bit to see if this is a fixable problem via something like proguard rules or if I have to see if it's possible to fix the underlying issue itself. |
@malcolmdeck Any update? I don't want to turn off full mode of R8 just because upgrading firebase sdk. |
same problem here. in firebase latest sdk. When using android.enableR8.fullMode=true . it's release version not working |
It's still reproducing with: |
Hello
|
@benjaminojanne Thanks but I think that will keep so many redundant thinks. |
@benjaminojanne It seems that you could specify to the path within auth library |
Any update on this without requiring a path? |
Make sure you have these lines:
Thanks #2124 (comment) |
+1 |
Didn't solve it for me |
thanks, it works |
This show as error in proguard file, any idea why? |
Is the other rule really necessary? Currently even with this line only in rule seems to solve the issue already. |
Now that Android Studio Flamingo with AGP 8.0 enabled R8 full mode by default, I'm facing the same issue. Is there any chance the auth library could provide R8 consumer rules in its artifact? |
@svenjacobs you can add these lines into proguard and it works. -keep public class com.google.firebase.** {;} |
@tekinarslan I only added this line to my R8 rules as mentioned here and it works for me.
|
In case a reproducer is needed, this is happening joreilly/Confetti (Pull Request to keep classes). |
Hi @LeandroLCD, not sure if this is the same issue. Could you file a new ticket with details from our template regarding your issue? Thanks! |
Hello, this is a particularly sneaky bug, a not nice at all pitfall that Google left on the path of developers AND users. How close is this to be fixed on Firebase side? |
I mean, it was reported back in 2020, and despite reports, part of Google itself pulled the trigger to make R8 full mode default in STABLE Android Gradle Plugin, and other part of Google still hasn't reacted… The consequences aren't great, please, prioritize this show-stopper. |
This has been fixed in https://firebase.google.com/support/release-notes/android#auth_v22-1-0 |
[READ] Step 1: Are you in the right place?
Y
[REQUIRED] Step 2: Describe your environment
[REQUIRED] Step 3: Describe the problem
Steps to reproduce:
If application is build with
android.enableR8.fullMode=true
then crash happens during login using google Sign-in.With setting
android.enableR8.fullMode=false
app runs fine.Stacktrace:
Relevant Code:
The text was updated successfully, but these errors were encountered: