reth icon indicating copy to clipboard operation
reth copied to clipboard

fix(transaction-pool): blob_tx_priority argument order to use (cap, current)

Open Snezhkko opened this issue 1 week ago • 1 comments
trafficstars

This change corrects the argument order passed to blob_tx_priority in BlobTransaction::new and BlobTransaction::update_priority to follow the documented fee_delta(max_tx_fee, current_fee) semantics. Previously, the arguments were inverted, effectively comparing current network fees against transaction caps, which could zero out priorities and break eviction ordering. The fix ensures deltas become negative when caps are below current fees, aligning with BlobOrd’s eviction logic and the filtering semantics used elsewhere in this module. Added unit tests assert exact priority values to prevent regressions in argument orientation.

Snezhkko avatar Nov 07 '25 09:11 Snezhkko