Skip to content

fix: clamp TrueBlockValue payout to relay max_bid_eth - #938

Open
Tyagiquamar wants to merge 1 commit into
flashbots:developfrom
Tyagiquamar:fix/max-bid-eth-true-block-value-700
Open

Tyagiquamar wants to merge 1 commit into
flashbots:developfrom
Tyagiquamar:fix/max-bid-eth-true-block-value-700

Conversation

@Tyagiquamar

Copy link
Copy Markdown

Summary

  • Read max_bid_eth from each relay''s submit config and clamp NewTrueBlockValueSlotBidder payout values before sealing bids.
  • Prevents funded coinbase balance from inflating constructed payouts above the operator-configured cap (relay submit still skipped over max, but payout tx was uncapped).

Test plan

  • cargo fmt --all --check (WSL, rbuilder workspace)
  • cargo test -p rbuilder --lib true_value_bidding_service -- --test-threads=2 (WSL; full rbuilder lib compile was in progress when PR opened; CI should run checks)
  • Added unit tests: max_bid_for_relay_set_requires_cap_on_every_relay, clamp_payout_respects_relay_max_bid

Fixes #700

Apply per-relay max_bid_eth from relay submit config when constructing
TrueBlockValue slot bids so funded coinbase balance cannot inflate payouts
above the operator cap.

Fixes flashbots#700
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.

max_bid_eth not parsed .. bids could exceed cap if coinbase funded

1 participant