-
-
Notifications
You must be signed in to change notification settings - Fork 270
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
MarshmallowNetworkObservingStrategy.java line 80 #462
Comments
Thanks for reporting this. I'll have a look on that. |
Hi there, Thanks for a wonderful library. Just following up on this issue. It's a known bug in Android 11, which was fixed in Android 12. However, the only workaround for it in Android 11 requires catching the exception when calling Please catch this exception by scheduling a periodic retry call to Cheers!
|
Hi 👋 +1 about the comment just above, I also encounter this crash on Android 11 devices. I'm catching the exception but if the library could handle it in a nicer way, that would be great 🙂 |
Hello, Thanks |
The bug is reproduce only on 11 android when call ReactiveNetwork.observeNetworkConnectivity(context)
https://android-review.googlesource.com/c/platform/frameworks/base/+/1758029/
In ExoPlayer it is fixed like this
google/ExoPlayer@2e21208
android.net.ConnectivityManager.registerNetworkCallback (ConnectivityManager.java:4564)
com.github.pwittchen.reactivenetwork.library.rx2.network.observing.strategy.MarshmallowNetworkObservingStrategy.observeNetworkConnectivity (MarshmallowNetworkObservingStrategy.java:80)
com.github.pwittchen.reactivenetwork.library.rx2.ReactiveNetwork.observeNetworkConnectivity (ReactiveNetwork.java:92)
com.github.pwittchen.reactivenetwork.library.rx2.ReactiveNetwork.observeNetworkConnectivity (ReactiveNetwork.java:73)
Could you please catch this exception in library?
Thanks.
The text was updated successfully, but these errors were encountered: