Skip to content
This repository has been archived by the owner on Sep 5, 2020. It is now read-only.

refactor sendTransaction error alerts #664

Open
luclu opened this issue May 18, 2016 · 4 comments · Fixed by #959
Open

refactor sendTransaction error alerts #664

luclu opened this issue May 18, 2016 · 4 comments · Fixed by #959

Comments

@luclu
Copy link
Contributor

luclu commented May 18, 2016

Currently "wrong password" error will show up for any node-related error.

bildschirmfoto 2016-07-09 um 13 45 23 kopie

We will try to catch the errors by string for now. But to ensure consistency and compatibility to other nodes these error codes should be implemented by the nodes.

update:
Following strings will be emitted by geth:

  • could not decrypt key with given passphrase (wrong password)
  • multiple keys match address (multiple copies of the same account)
  • Insufficient funds for gas * price + value (not enough ether in etherbase account to invoke contract transaction)
  • Unable to connect to socket: timeout (node terminated, etc; used to be CONNECTION ERROR as in Mist's current code)

fixed with PR #959

Additionally there should be a warning if a transaction should deplete the etherbase account to a balance that would not allow to execute transfers from contract wallets.

duplicates:
#368
#866
#869
#875
#923
#926
#948

@luclu luclu changed the title Improve error msg of contract execution when balance of coinbase is too low Improve error msg of failed contract execution when balance of coinbase is too low May 18, 2016
@luclu luclu added this to the 0.8 milestone Jun 17, 2016
@luclu luclu self-assigned this Jul 9, 2016
@luclu luclu modified the milestones: 0.8.1, 0.8 Jul 9, 2016
@luclu luclu changed the title Improve error msg of failed contract execution when balance of coinbase is too low Improve error msg ("wrong password") of failed contract execution when balance of coinbase is too low Jul 11, 2016
@luclu luclu changed the title Improve error msg ("wrong password") of failed contract execution when balance of coinbase is too low refactor sendTransaction error alerts Jul 17, 2016
@luclu luclu removed their assignment Mar 19, 2017
@evertonfraga evertonfraga reopened this Aug 14, 2017
@crypt61
Copy link

crypt61 commented Dec 5, 2017

I am getting the error "Insufficient funds for gas * price + value”. However, I have a balance of 0.090948
ETH, and the "max total" of the transaction is 0.076000. That seems to be plenty of funds to complete the transaction. What am I missing? Thank you for your help.

@Sharkhan
Copy link

Sharkhan commented Dec 5, 2017

Hello, have a same issue insufficient funds for gas * price + value. Any solution?
Thank you

@crypt61
Copy link

crypt61 commented Dec 5, 2017

Here is the response from MetaMask...

Hello,

There is currently a combination of both a backlog in pending transactions on the network as well as rising competition for gas prices with CryptoKitties. Unfortunately this is causing some transactions with lower gas prices to take longer to process or even fail. Unfortunately there is little to be done about this besides waiting for any pending transactions to go through or show as failed/rejected. This waiting period can take up to 24 hours during congested network times such as this.

For your refernece in using gas with Dapps such as CryptoKitties, Ether Delta and others: https://support.metamask.io/kb/article/25-transaction-submission-best-practices-on-etherdelta-and-dapps/

@ghost
Copy link

ghost commented Feb 8, 2018

Could "wrong password" error show up upon importing a pre-sale wallet due to any node-related errors?
I have had trouble syncing nodes. But then I get "wrong password" when I import my pre-sale wallet through Kraken, Mist ect too

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants