Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tx failed, transaction will be removed #519

Closed
ethers opened this issue Mar 18, 2015 · 11 comments
Closed

tx failed, transaction will be removed #519

ethers opened this issue Mar 18, 2015 · 11 comments

Comments

@ethers
Copy link
Member

ethers commented Mar 18, 2015

I captured loglevel5:
2015/03/17 19:50:59 [MINER] TX (4146cbcd) failed. Transaction will be removed

http://pastebin.com/Vg5CzcSG
The failed is at the bottom.

When the tx is run again, it succeeds.

@ethers
Copy link
Member Author

ethers commented Mar 18, 2015

This was found on develop commit 3ea99f9

@ethers
Copy link
Member Author

ethers commented Mar 20, 2015

With PR #543 here's the error:
[MINER] TX (c05b349e) failed, will be removed: Transaction w/ invalid nonce (7 / 8)
Looks related to #317 ?

@ethers
Copy link
Member Author

ethers commented Mar 20, 2015

In case helpful, here is a very short, standard log showing 2 instances of invalid nonce: http://pastebin.com/yMPZrFyj

@ethers
Copy link
Member Author

ethers commented Mar 21, 2015

519 could be more serious; I added: A tx that transferred ether from one account to another account, was removed due to invalid nonce:
2015/03/20 20:51:42 [MINER] TX (b54b72e9) failed, will be removed: Transaction w/ invalid nonce (16 / 17)
However, the balances of the accounts were NOT reverted. If tx failed, I would expect that any state changes it causes, would also be reverted.

I had sent similar tx via frontend+ethereum.js and didn't see the balance updates, usually after
[MINER] committing new work with 1 txs the balance would update via jsonrpc. Now, have to wait until the next [MINER] committing new work with 0 txs to see the balance update. It was these balance updates that I misattributed as being caused by the tx invalid nonce above.

@obscuren
Copy link
Contributor

Do you have sample sendTransaction data I can use to replicate this? Or does this happen with the most basic Txs?

@obscuren
Copy link
Contributor

I've tried to fire some transactions myself but it I can't seem to reproduce the error you're having. Could you please provide me with same samples? Could you also retry using the conversion branch?

@ethers
Copy link
Member Author

ethers commented Mar 21, 2015

I will try on conversion branch.
For now, to repro consistently I use pyepm https://github.com/etherex/pyepm and this yaml
https://github.com/ethers/dapp-bin/blob/btcrelay/btcrelay/zGo-testBulkDeploy.yaml

With Go cli running, it is pyepm -a <your Go unlocked account> zGo-testBulkDeploy.yaml

@ethers
Copy link
Member Author

ethers commented Mar 21, 2015

On conversion branch, looks like it still happens [MINER] TX (39b66ba4) failed, will be removed: Transaction w/ invalid nonce (14 / 0)

Here's more log which has quite a bit of data, including tx data, and hopefully helps:
http://pastebin.com/vAXbA7Tk

@obscuren
Copy link
Contributor

I think I've found the issue and fixed in c28116c

Could you try again?

@ethers
Copy link
Member Author

ethers commented Mar 24, 2015

looks fixed on e88a29a

@samoclay
Copy link

Hey guys, I got something similar on a transaction I made on a recent ICO
https://etherscan.io/tx/0x6ed7aca844244dca2d6d34f8bdb10aaf208971c1164ff08aa6d7c4b9abb00ddc
Im not sure what this means though?

nolash pushed a commit to nolash/go-ethereum that referenced this issue May 11, 2018
swarm/api: Support If-None-Match conditional HTTP requests
tanishqjasoria pushed a commit to tanishqjasoria/go-ethereum that referenced this issue Oct 31, 2023
tanishqjasoria pushed a commit to tanishqjasoria/go-ethereum that referenced this issue Oct 31, 2023
* Revert "feat: use light_mode = false for block ccc (ethereum#519)"

This reverts commit a51434d.

* bump version
s1na pushed a commit to s1na/go-ethereum that referenced this issue Dec 2, 2024
implement EIP-2929 for EIP-1559
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants