Pairing fails with BLE device Feit LED bulb

[bdraco]

https://www.amazon.com/dp/B07J2FP389?psc=1&ref=ppx_yo2ov_dt_b_product_details

Feit Electric OM60/SW/HK 60W Equivalent A19 Smart, Works with Apple HomeKit and Siri Voice Control, No Hub Required LED Light Bulb, 4.4" H x 2.4" D, 2700K Soft White

‎OM60/SW/HK

2022-08-07 12:23:04.157 DEBUG (MainThread) [homeassistant.components.homekit_controller.config_flow] Pairing authentication failed
Traceback (most recent call last):
  File "/Users/bdraco/home-assistant/homeassistant/components/homekit_controller/config_flow.py", line 464, in async_step_pair
    pairing = await self.finish_pairing(code)
  File "/Users/bdraco/home-assistant/venv/lib/python3.10/site-packages/aiohomekit/controller/ble/client.py", line 61, in _async_wrap
    return await func(*args, **kwargs)
  File "/Users/bdraco/home-assistant/venv/lib/python3.10/site-packages/aiohomekit/controller/ble/discovery.py", line 147, in finish_pairing
    pairing = await drive_pairing_state_machine(
  File "/Users/bdraco/home-assistant/venv/lib/python3.10/site-packages/aiohomekit/controller/ble/client.py", line 195, in drive_pairing_state_machine
    request, expected = state_machine.send(TLV.decode_bytes(response))
  File "/Users/bdraco/home-assistant/venv/lib/python3.10/site-packages/aiohomekit/protocol/__init__.py", line 222, in perform_pair_setup_part2
    handle_state_step(response_tlv, TLV.M4)
  File "/Users/bdraco/home-assistant/venv/lib/python3.10/site-packages/aiohomekit/protocol/__init__.py", line 87, in handle_state_step
    error_handler(tlv_dict[TLV.kTLVType_Error], f"step {expected_state}")
  File "/Users/bdraco/home-assistant/venv/lib/python3.10/site-packages/aiohomekit/protocol/__init__.py", line 61, in error_handler
    raise AuthenticationError(stage)
aiohomekit.exceptions.AuthenticationError: step bytearray(b'\x04')

[bdraco]

I've been messing with this for hours and I haven't been able to figure out why it rejects the code. Everything looks right. I went through the srp code line by line and it all seems right.

Pairs fine with iOS though

[bdraco]

After messing with it for hours, I changed the MTU to 100 it pairs

[bdraco]

I also had to increase the read timeout to 20s from 10s on macos

[Jc2k]

Oh god. I love that the error is an auth error.

[bdraco]

So we need to do three things

• Get bleak to know about the MTU in the char data BlueZ client._acquire_mtu() raises RuntimeError on many devices hbldh/bleak#925 -- solved via Add support for bleak max_write_without_response_size and current bleak MTU #141
• Raise the timeout on mac yet again for reads corebluetooth read_characteristic timeout is too low hbldh/bleak#926 Adjust default timeout for read_gatt_char() with CoreBluetooth to 20s hbldh/bleak#928
• Lower the MTU to 100 here until we get bleak to know about it Drop mtu auto discovery #139

Sadly in the mean time thats going to make reconnects even slower but hbldh/bleak#923 will speed that up

[Jc2k]

Sounds good 👍

[Djelibeybi]

Aside: Feit Electric just saved bought LIFX. That's not relevant to this issue, but I thought it was interesting enough to share.

[bdraco]

The bleak PR to raise the timeout is merged. Nothing more to track here. We will get the fix for macos when we bump bleak in HA