fix(core/txpool): clarify tx validation error messages #29081#2163
fix(core/txpool): clarify tx validation error messages #29081#2163gzliudan wants to merge 1 commit intoXinFinOrg:dev-upgradefrom
Conversation
Adjust txpool validation error wording to report actual and minimum values consistently. - intrinsic gas error now reports: gas <actual>, minimum needed <required> - underpriced tip error now reports: gas tip cap <actual>, minimum needed <required> No transaction validation logic is changed in this commit; only error message text is updated.
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Pull request overview
This PR standardizes the wording of transaction validation error messages in core/txpool/validation.go to consistently report the actual value first and the minimum required value second, using the format <field> <actual>, minimum needed <required>.
Changes:
- Reworded intrinsic gas error to report
gas <actual>, minimum needed <required>(wasneeded <required>, allowed <actual>). - Reworded underpriced tip errors (both for special transactions and normal transactions) to report
gas tip cap <actual>, minimum needed <required>(wastip needed <required>, tip permitted <actual>).
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Proposed changes
Adjust txpool validation error wording to report actual and minimum values consistently.
intrinsic gas error now reports: gas , minimum needed
underpriced tip error now reports: gas tip cap , minimum needed
No transaction validation logic is changed in this commit; only error message text is updated.
Ref: ethereum#29081
Types of changes
What types of changes does your code introduce to XDC network?
Put an
✅in the boxes that applyImpacted Components
Which parts of the codebase does this PR touch?
Put an
✅in the boxes that applyChecklist
Put an
✅in the boxes once you have confirmed below actions (or provide reasons on not doing so) that