mozilla-magnet / magnet-scanner-android

An Android library that scans for URLs around you
3 stars 6 forks source link

Crash opening notification #16

Closed wilsonpage closed 7 years ago

wilsonpage commented 7 years ago

Moved from https://github.com/mozilla-magnet/magnet-client/issues/283

java.lang.IllegalStateException: GoogleApiClient is not connected yet.
    at com.google.android.gms.internal.zzrb.zzb(Unknown Source)
    at com.google.android.gms.internal.zzrf.zzb(Unknown Source)
    at com.google.android.gms.internal.zzrd.zzb(Unknown Source)
    at com.google.android.gms.location.internal.zzd.removeLocationUpdates(Unknown Source)
    at org.mozilla.magnet.scanner.geolocation.ScannerGeolocation.stop(ScannerGeolocation.java:83)
    at org.mozilla.magnet.scanner.MagnetScanner.stop(MagnetScanner.java:112)
    at org.mozilla.magnet.MagnetScannerReact.stop(MagnetScannerReact.java:52)
    at java.lang.reflect.Method.invoke(Native Method)
    at com.facebook.react.bridge.BaseJavaModule$JavaMethod.invoke(BaseJavaModule.java:318)
    at com.facebook.react.cxxbridge.JavaModuleWrapper.invoke(JavaModuleWrapper.java:158)
    at com.facebook.react.bridge.queue.NativeRunnable.run(Native Method)
    at android.os.Handler.handleCallback(Handler.java:739)
    at android.os.Handler.dispatchMessage(Handler.java:95)
    at com.facebook.react.bridge.queue.MessageQueueThreadHandler.dispatchMessage(MessageQueueThreadHandler.java:31)
    at android.os.Looper.loop(Looper.java:148)
    at com.facebook.react.bridge.queue.MessageQueueThreadImpl$3.run(MessageQueueThreadImpl.java:196)
    at java.lang.Thread.run(Thread.java:818)
wilsonpage commented 7 years ago

@arcturus r?

Can you check this branch out and yarn link magnet-scanner-android into magnet-client to see if this fixes the bug your found. I can't seem to reproduce at my end but looking at the stack trace, this should help protect.