ruuvi / ruuvi.firmware.c

Ruuvi Firmware version 3. Built on top of Nordic SDK 15, uses both Ruuvi and external repositories as submodules. In Beta, no breaking changes are intended but may be done if absolutely necessary
BSD 3-Clause "New" or "Revised" License
52 stars 39 forks source link

nRF reboots if trying to pair using Android's Bluetooth manager #281

Open laurijamsa opened 3 years ago

laurijamsa commented 3 years ago

If I try to pair with Android, RuuviTag reboots, measurement sequence number resets and history gets lost.

image

Tried with Samsung Galaxy S21 Ultra and Lenovo tablet.

Defective firmware versions: 3.30.2 and 3.30.3 (and possibly older ones).

ojousima commented 3 years ago

Possibly fixed in #285, bonding was disabled in config. Needs verification

niksiboxi commented 2 years ago

The RuuviTag crashes with error log ruuvi_nrf5_sdk15_communication_ble_gatt.c:365 FATAL: INVALID_STATE

niksiboxi commented 2 years ago

Possible solution is to handle the Android pairing process with the peer manager, hence the BLE_GAP_EVT_SEC_PARAMS_REQUEST is not needed.

ojousima commented 2 years ago

@markoaamunkajo please check if this is fixed in 3.31.2-RC1

markoaamunkajo commented 2 years ago

@ojousima Still reboots 3.31.2-RC1 tag rev B7 and B8.

ojousima commented 2 years ago

@niksiboxi Can you take a look?