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

Invalidate txs with below base-fee-floor gas-price in txpool #292

Open
wants to merge 7 commits into
base: celo11
Choose a base branch
from

Conversation

ezdac
Copy link

@ezdac ezdac commented Dec 5, 2024

Fixes #291

Before, the transaction pool accepted transactions that have a gas-price below the configured base-fee-floor, which would never be executed.

Now, the transaction pool rejects those transactions immediately - giving the submitting user an immediate response and better UX.

Copy link

@palango palango left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine, but needs some additional checks to avoid test failures.

And please have a look and see if it's possible to add a test for this.

core/txpool/celo_validation.go Outdated Show resolved Hide resolved
@ezdac ezdac force-pushed the ezdac/fix-291 branch 2 times, most recently from 0020729 to 55c18e1 Compare December 12, 2024 15:12
@ezdac ezdac changed the base branch from celo10 to celo11 December 20, 2024 09:41
Fixes #291

Before, the transaction pool accepted transactions that have a
gas-price below the configured base-fee-floor, which would never be
executed.

Now, the transaction pool rejects those transactions immediately -
giving the submitting user an immediate response and better UX.
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

Successfully merging this pull request may close these issues.

Node not rejecting txs with gas price less than 25 Gwei
2 participants