Add BLE support with 'bleak'

README.md

@@ -135,6 +135,7 @@ $ ledgerctl -v run Bitcoin
 => e0d8000007426974636f696e
 <= 9000
 ```
+
 ### Using BLE
 
 BLE scanning is disabled by default. It can be activated by setting an environment variable named `LEDGER_USE_BLE`.
@@ -159,4 +160,3 @@ And executed with:
 ```console
 pre-commit run --all-files
 ```
-

ledgerwallet/client.py

@@ -164,6 +164,7 @@ class NoLedgerDeviceException(Exception):
 
 class LedgerClient(object):
     def __init__(self, device=None, cla=0xE0, private_key=None):
+        self.device = None
         if device is None:
             devices = enumerate_devices()
             if len(devices) == 0:
@@ -184,7 +185,8 @@ def __del__(self):
         self.close()
 
     def close(self):
-        self.device.close()
+        if self.device is not None:
+            self.device.close()
 
     def raw_exchange(self, data: bytes) -> bytes:
         LOG.debug("=> " + data.hex())

ledgerwallet/transport/__init__.py

@@ -1,7 +1,7 @@
 from contextlib import contextmanager
 
-from .device import Device
 from .ble import BleDevice
+from .device import Device
 from .hid import HidDevice
 from .tcp import TcpDevice
 

ledgerwallet/transport/ble.py

@@ -1,8 +1,9 @@
 import asyncio
 import os
+from typing import List
+
 from bleak import BleakClient, BleakScanner
 from bleak.exc import BleakError
-from typing import List
 
 HANDLE_CHAR_ENABLE_NOTIF = 13
 HANDLE_CHAR_WRITE = 16
@@ -13,7 +14,7 @@
 
 
 async def ble_discover():
-    devices = await BleakScanner.discover(timeout=1.)
+    devices = await BleakScanner.discover(timeout=1.0)
     return devices