Merge #718: Improve error handle for Ledger devices

0d5412a Improve error handle for `Ledger` devices (Bruno Garcia)

Pull request description:

  Discovered it while reviewing [#bitcoin/24313](bitcoin/bitcoin#24313). The `ledger_exception` function was not dealing with `ApduException` exceptions. It's basically similar to `BTChipException` and we can use its "error code" to return friendly error messages.

  e.g for `displayaddress`:

  **before**:
  `{"error": "Exception: invalid status 0x6985", "code": -13}`
  **after**:
  `{"error": "display_singlesig_address canceled", "code": -14}`

ACKs for top commit:
  achow101:
    ACK 0d5412a

Tree-SHA512: f4dbe57e5f41c67a19ce6e9c0f4b4af8184ff360cfc3709f4fb15f55014906c0a6c04a869a9315083aac18175dd8b921c022d18826d3e1f31d4bdb4a736b6171

hwilib/devices/ledger.py

@@ -39,6 +39,7 @@
     LegacyClient,
     TransportClient,
 )
+from .ledger_bitcoin.client_base import ApduException
 from .ledger_bitcoin.exception import NotSupportedError
 from .ledger_bitcoin.wallet import (
     MultisigWallet,
@@ -118,6 +119,18 @@ def check_keypath(key_path: str) -> bool:
     AddressType.LEGACY: 3,
 }
 
+def handle_chip_exception(e, func_name: str) -> bool:
+    if e.sw in bad_args:
+        raise BadArgumentError('Bad argument')
+    elif e.sw == 0x6F00: # BTCHIP_SW_TECHNICAL_PROBLEM
+        raise DeviceFailureError(e.message)
+    elif e.sw == 0x6FAA: # BTCHIP_SW_HALTED
+        raise DeviceConnectionError('Device is asleep')
+    elif e.sw in cancels:
+        raise ActionCanceledError('{} canceled'.format(func_name))
+    else:
+        raise e
+
 def ledger_exception(f: Callable[..., Any]) -> Any:
     @wraps(f)
     def func(*args: Any, **kwargs: Any) -> Any:
@@ -126,16 +139,10 @@ def func(*args: Any, **kwargs: Any) -> Any:
         except ValueError as e:
             raise BadArgumentError(str(e))
         except BTChipException as e:
-            if e.sw in bad_args:
-                raise BadArgumentError('Bad argument')
-            elif e.sw == 0x6F00: # BTCHIP_SW_TECHNICAL_PROBLEM
-                raise DeviceFailureError(e.message)
-            elif e.sw == 0x6FAA: # BTCHIP_SW_HALTED
-                raise DeviceConnectionError('Device is asleep')
-            elif e.sw in cancels:
-                raise ActionCanceledError('{} canceled'.format(f.__name__))
-            else:
-                raise e
+            handle_chip_exception(e, f.__name__)
+        except ApduException as e:
+            e.message = e.data.decode("utf-8")
+            handle_chip_exception(e, f.__name__)
     return func
 
 # This class extends the HardwareWalletClient for Ledger Nano S and Nano X specific things