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

bug: no way to update public transaction after submitting transaction with gas below intrinsic limit #504

Open
annamcallister opened this issue Jan 7, 2025 · 1 comment
Labels
bug Something isn't working

Comments

@annamcallister
Copy link
Contributor

What happened?

I submitted a public transaction with a gas limit below the intrinsic limit, which was rejected by the chain. This meant the nonce was not used and all subsequent transactions from that signing key with higher nonces became stuck in the mem pool.

What did you expect to happen?

To have a way to continue submit transactions. I think what that means is being able to update the transaction to increase the gas limit, or to nullify the transaction.

We could potentially guard against this issue by estimating the intrinsic gas, but this would an additional rpc call to the node and since it's an estimate the issue could still occur, so this is probably an enhancement if we want it, and less important than the ability to update a transaction.

How can we reproduce it (as minimally and precisely as possible)?

Submit a public transaction with a very low gas limit

Anything else we need to know?

No response

OS version

# On Linux:
$ cat /etc/os-release
# paste output here
$ uname -a
# paste output here

# On Windows:
C:\> wmic os get Caption, Version, BuildNumber, OSArchitecture
# paste output here
@annamcallister annamcallister added the bug Something isn't working label Jan 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants
@annamcallister and others