Fix crash for AccessToken tracker #699
Fix crash for AccessToken tracker #699
Conversation
This PR fixes Crash when handling access token by tracking access token when react context is ready
@mikehardy, @luancurti sry for pulling you into this PR, can you please try to test?? |
@chakrihacker I've tested your code and I release a new version of my app with your fix. Great Job! 🎉 👏 |
I won't have time, I'm sorry but kudos to @luancurti for giving it a go and for doing the PR in the first place @chakrihacker 💪 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work everyone, thanks!
Released as 1.1.2 |
Does this fix, happened on 1.1.1
|
@UberMC yes I believe it fixes exactly that. My related commit in my project:
No crashes since integrating the patch+1.1.1 or using 1.1.2 clean from upstream |
@mikehardy still happening here on 1.1.2, lots of crashes on Google Play Console from production users. This is the report: java.lang.RuntimeException: Caused by: java.lang.NullPointerException: |
This seems like a different issue. Also I don't see any code related to the react-native bindings, looks like a bug in the facebook sdk. Maybe updating the version could fix it. |
@janicduplessis thanks for replying. Weird enough I'm using react-native-fbsdk 1.1.2 (latest) and RN 0.61.5, followed step by step for autolink config, works well on iOS but Google Play Console is showing hundreds of crashes daily on production from users. Opened an issue for this but not sure what is happening ( #750 ). Seems like some people also had the same issue ( facebook/react-native#24498 (comment) ). |
I am also facing this issue. I'm using RN 0.61.5 and react-native-fbsdk 1.1.2. java.lang.NullPointerException Attempt to invoke virtual method 'void com.facebook.marketing.ViewIndexer.unschedule()' on a null object reference |
@Hamawis v2.0.0 is out, reporting issues on old versions and closed PRs is likely to result in not a thing happening |
This PR fixes Crash when handling access token by tracking access token when react context is ready
Fixes #675