gnosis / MultiSigWallet

Allows multiple parties to agree on transactions before execution.
https://github.com/gnosis/MultiSigWallet/releases
GNU Lesser General Public License v3.0
1.08k stars 762 forks source link

[bug] timeout when confirming transaction #95

Open florianlenz opened 6 years ago

florianlenz commented 6 years ago

Hi,

we are playing around with the gnosis multi sig wallet. When we try to confirm a transaction with ledger we almost have no time to read the address, eth and gas feed since we are getting a timeout from the gnosis wallet. Could you please increase the timeout?

Thank you.

denisgranha commented 6 years ago

Yeah would be nice, I remember it was harcoded in one of the dependent libraries, will try to include this in the next release

florianlenz commented 6 years ago

Thank you. Tell me if I can support you.

jeremyschlatter commented 5 years ago

It looks like this might be fixed upstream, as of https://github.com/LedgerHQ/ledgerjs/pull/67

Unfortunately it also looks to me like you can't simply bump the version number, because the relevant package hasn't been updated in ten months. Ledger has a bunch of other npm packages that are up-to-date, but switching to them might require a non-trivial code rewrite.

I encountered this issue today and it was pretty frustrating.

denisgranha commented 5 years ago

one alternative is use the electron version (desktop build) it uses another ledger library that doesn’t have tiemout