-
Notifications
You must be signed in to change notification settings - Fork 257
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
NullPointerException in FB.java Line 378 #18
Comments
I think what has happened here is that the unity activity has died when you come back from login. We then try to access to get the keyhash and crash. I'll look at fixing this in the next release
|
I've done the same as Patrick and added the NullPointerException, I've not personally been able to recreate the issue but looking through the Google Play Developer Console it's our number one crash issue. I've subscribe to this issue but is there an eta on the next version? |
I see a lot of these as well for my users... |
Is this fixed now? I see in the 7.5.0 changelog there's a Fixed: NPE on Android when Unity activity was killed during login flow. |
Hope this is fixed in with V7.5 |
This should have been fixed in 7.5.0 |
@swiese , we are also seeing this issue in production release, We are using FB SDK 7.4.0, is this fixed in 7.5.0? |
After deploying to our customers we found a decent percentage of users running into NullPointerExceptions when logging in.
Stack Trace:
com.facebook.unity.FB.getKeyHash (FB.java:378)
com.facebook.unity.FBLogin.login (FBLogin.java:92)
com.facebook.unity.FBLogin.loginWithReadPermissions (FBLogin.java:42)
com.facebook.unity.FBUnityLoginActivity.onCreate (FBUnityLoginActivity.java:40)
android.app.Activity.performCreate (Activity.java:5372)
dalvik.system.NativeStart.main (NativeStart.java)
We were able to fix locally by adding NullPointerException to the list of caught exceptions near line 378 in FB.java but this potentially hides something that should be fixed.
Thank you!
The text was updated successfully, but these errors were encountered: