Callback onConnectionStateChange after disconnect can put BleManager in inconsistent state about android-ble-library HOT 5 CLOSED

I have now, the fix seems to have solved the problem. I have run the same test with the new code 50 times, no issues, connection works beautifully. Thanks again!

from android-ble-library.

Thank you for the feedback. I can't do anything about the extra onConnectionStateChangeSafe, as it's being called by the system, but the lib should not get stack when it's called. Expect a fix soon.

from android-ble-library.

From my experience this happens when Android receives a notification after the disconnect() was called. First it notifies you that it's disconnected, but when it gets the notification it's like "hmm.. I'm still connected, so here's your callback with connected state".
What phone are you using for testing and how often a) you send notifications from the peripheral b) you experience this issue? You are not using autoConnect, right?

There are 2 solutions that can be done here:

1. In onConnectionStateChangedSafe, where status == BluetoothGatt.GATT_SUCCESS && newState == BluetoothProfile.STATE_CONNECTED, and mBluetoothDevice == null I may call gatt.close() before setting mConnected to true and pretend that it never happened. After all, you called disconnect(), so the desire effect is to be disconnected.
2. I may try to recover to a state from before disconnection: mBluetoothDevice = gatt.getDevice(), mBluetoothGatt = gatt , and mGattCallbacks = this. But I'd have to register the receivers again, as close() was called before.

Looks like 1. is a better solution.

from android-ble-library.

Thanks for the quick fix! Here's the info you asked for:
Phone used for testing: Google Pixel with Android 7.1.2.
a. When doing this test, constantly. The peripheral is trying to stream data by repeatedly writing a characteristic.
b. Fairly frequently, maybe after 20% of the disconnects (rough approximation). No, I am not using autoConnect.

from android-ble-library.

By any chance, did you try the version from referenced PR?

from android-ble-library.