Improve error handle for Ledger devices

[brunoerg]

Discovered it while reviewing #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}

[brunoerg]

cc: @Sjors

[Sjors]

@bigspider is there a complete list of (Bitcoin app) error codes published somewhere?

[bigspider]

@bigspider is there a complete list of (Bitcoin app) error codes published somewhere?

This mapping is currently as good as it gets. The IncorrectDataError (SW 0x6A80) is the most common error and it's returned for way too many distinct error situations; hopefully I find some time to improve it with more fine-grained error results.

[achow101]

ACK 0d5412a

[Sjors]

0x5515 is what I get when the app is in screensaver mode (Ledger Nano X, BTC app 2.1.3)

[bigspider]

0x5515 is what I get when the app is in screensaver mode (Ledger Nano X, BTC app 2.1.3)

Indeed, that situation used to be managed by the app directly, while now it's handled in the app sdk, so the error code changed. I'll add the error code to the client libraries, thanks!