mikeshultz / ledger-eth-lib

Library to interface with Ethereum app on Ledger hardware wallets
https://ledgereth.readthedocs.io/
MIT License
19 stars 9 forks source link

Issue with comms.py #27

Closed ThibltJ closed 2 years ago

ThibltJ commented 2 years ago

When I want to retrieve the accounts from the ledger (v0.7.1), I have this error :

File "../lib/python3.8/site-packages/ledgereth/comms.py", line 168, in init_dongle
    if not is_usable_version(DONGLE_CONFIG_CACHE):
NameError: name 'DONGLE_CONFIG_CACHE' is not defined

The workaround that I found is the initialisation of the global variables

DONGLE_CACHE: Dongle = None
DONGLE_CONFIG_CACHE: bytes = None

Let me know if I can give you more information in order to help you

mikeshultz commented 2 years ago

Thanks for the report. Looking into it.

mikeshultz commented 2 years ago

Fix went out with v0.7.2.