Keepkey and Electrum?

wndsnb

New Member
Aug 20, 2017
8
0
1
45
Has anyone managed to get a Keepkey wallet working with Electrum-Dash?

When I try to create a wallet with my Keepkey, I get this error:

 

akhavr

Active Member
Oct 11, 2014
759
379
133
I don't have a keepkey (looks like I have to buy one...)

Please, make sure that you have the complete installation, in particular whether the hid module is installed correctly. I guess you're under linux, right?
 

akhavr

Active Member
Oct 11, 2014
759
379
133
Is this happening only with keepkey wallet, or with regular wallets too?
 

Dariusz Myśliwiec

New Member
Jul 8, 2017
11
3
3
28
I got this error just while using the Electrum-Dash.
EDIT: I'll try to elaborate on it and look for some fixes to it, but don't promise anything.
 
Last edited:

tungfa

Administrator
Dash Core Team
Moderator
Foundation Member
Masternode Owner/Operator
Apr 9, 2014
8,890
6,717
1,283
I got this error just while using the Electrum-Dash.
EDIT: I'll try to elaborate on it and look for some fixes to it, but don't promise anything.
did u try it with DMT App ? (no electrum but keepkey direct ) ?
 

Dariusz Myśliwiec

New Member
Jul 8, 2017
11
3
3
28
did u try it with DMT App ? (no electrum but keepkey direct ) ?
Didn't try this tool yet, since I thought it's related to MN operations. The problem I have is to retrieve the accounts to Electrum and do regular spending. If DMT have some module for KeepKey, I'll try to verify if it works fine and let you know, so the working code can be copied from DMT to Electrum.
 

tungfa

Administrator
Dash Core Team
Moderator
Foundation Member
Masternode Owner/Operator
Apr 9, 2014
8,890
6,717
1,283
Didn't try this tool yet, since I thought it's related to MN operations. The problem I have is to retrieve the accounts to Electrum and do regular spending. If DMT have some module for KeepKey, I'll try to verify if it works fine and let you know, so the working code can be copied from DMT to Electrum.
my bad
sorry thought we are talking MN here
don't they have their own wallet as trezor.wallet ?
 

Dariusz Myśliwiec

New Member
Jul 8, 2017
11
3
3
28
No worries :)
They do have their own web wallet, but the upgrades are slow (eg. for Dash it still has the higher fees for transactions). And the UI is pretty basic + no way to make multiple payments in one transaction.
 
  • Like
Reactions: tungfa

Dariusz Myśliwiec

New Member
Jul 8, 2017
11
3
3
28
For now I can confirm that DMT works fine with my KeepKey on latest stable release.
Didn't have time to confirm more, but seems like the replacement of support library for KeepKey should fix this problem.
 

Dariusz Myśliwiec

New Member
Jul 8, 2017
11
3
3
28
Seems like I found the issue and I'm happy to provide a fix for it. It works for me at least. Because I cannot create new issue in the offficial GitHub for this project (https://github.com/dashpay/electrum-dash), I'm providing the fix here. I hope somebody can commit the solution to the repository and rebuild the sources:
In plugins/keepkey/keepkey.py there are missing lines at the end of the file, within KeepKeyPlugin class:
Code:
    def hid_transport(self, pair):
        from keepkeylib.transport_hid import HidTransport
        return HidTransport(pair)

    def bridge_transport(self, d):
        raise NotImplementedError('')
Please let me know if somebody will deal with it, or I can contribute somehow.

Edit: Seems like the code is already there, but the binaries for Windows don't have the fragment I've mentioned, so it just have to be recompiled from sources.
 
Last edited:
  • Like
Reactions: UdjinM6 and tungfa

tungfa

Administrator
Dash Core Team
Moderator
Foundation Member
Masternode Owner/Operator
Apr 9, 2014
8,890
6,717
1,283
Seems like I found the issue and I'm happy to provide a fix for it. It works for me at least. Because I cannot create new issue in the offficial GitHub for this project (https://github.com/dashpay/electrum-dash), I'm providing the fix here. I hope somebody can commit the solution to the repository and rebuild the sources:
In plugins/keepkey/keepkey.py there are missing lines at the end of the file, within KeepKeyPlugin class:
Code:
    def hid_transport(self, pair):
        from keepkeylib.transport_hid import HidTransport
        return HidTransport(pair)

    def bridge_transport(self, d):
        raise NotImplementedError('')
Please let me know if somebody will deal with it, or I can contribute somehow.

Edit: Seems like the code is already there, but the binaries for Windows don't have the fragment I've mentioned, so it just have to be recompiled from sources.
@akhavr
i believe this one if for u
tx
 
  • Like
Reactions: Dariusz Myśliwiec

akhavr

Active Member
Oct 11, 2014
759
379
133
Seems like I found the issue and I'm happy to provide a fix for it. It works for me at least. Because I cannot create new issue in the offficial GitHub for this project (https://github.com/dashpay/electrum-dash), I'm providing the fix here. I hope somebody can commit the solution to the repository and rebuild the sources:
In plugins/keepkey/keepkey.py there are missing lines at the end of the file, within KeepKeyPlugin class:
Code:
    def hid_transport(self, pair):
        from keepkeylib.transport_hid import HidTransport
        return HidTransport(pair)

    def bridge_transport(self, d):
        raise NotImplementedError('')
Please let me know if somebody will deal with it, or I can contribute somehow.

Edit: Seems like the code is already there, but the binaries for Windows don't have the fragment I've mentioned, so it just have to be recompiled from sources.
Thanks, this code will be included in our 2.9.3 release.