autoConnect=true attempts to reconnect even after invoking disconnect() about android-ble-library HOT 8 CLOSED

This fixes the issue for now, but it's not pretty

This is a wrapper that holds the BleManager class

public void connect() {
    forceDisconnect.set(false);
    bleManager.setGattCallbacks(this);

    try {
      bleManager.connect(bluetoothDevice).await();
    } catch (RequestFailedException | DeviceDisconnectedException | BluetoothDisabledException e) {
      if (forceDisconnect.get()) {
        bleManager.setGattCallbacks(this);

        bleManager.close();
      }

      e.printStackTrace();
    }
  }

public void disconnect() {
    forceDisconnect.set(true);

    bleManager.disconnectWithoutReconnect();
  }

from android-ble-library.

Hi, I'll look into it. Thanks for the report!

from android-ble-library.

Hi @maragues-kolibree,
I was thinking about what should actually happen in this case.
First of all, the correct way of closing pending connection attempt is by setting a timeout. This may be set wither using enqueue(int) or await(int). The PR makes sure the connection gets cancelled in such case.
Also, the lib won't start another connection if the first one failed dues to a timeout.

from android-ble-library.

If you release a new beta, I can test next Monday

from android-ble-library.

I released beta3.

from android-ble-library.

Fixed in beta4 :)

from android-ble-library.

It's fixed, congrats and thanks!!

It's beta3, not beta4, just to keep track for the future.

from android-ble-library.

After this change, is there a way to invoke connect in a way that it keeps trying to reconnect after an error?

I know we can do it manually, just wondering.

from android-ble-library.