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
|
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.
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.
7e1d164 to
5ce0d09
Compare
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