-
Notifications
You must be signed in to change notification settings - Fork 19
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
fix(auction): Adding extra check on bundler timeouts #156
Conversation
bidTimeout := bidInfo.Timeout | ||
|
||
for index, bundleTimeout := range bidInfo.TransactionTimeouts { | ||
signers := bidInfo.Signers[index] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: comment here would be nice, spent some time tryna figure out what this did (check if the bidder is signer of a sub-tx). May also attribute confusion to me being dumb 🔢
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
* validate bundler txs * nits * lint (cherry picked from commit 339b927) # Conflicts: # tests/app/app.go # tests/integration/block_sdk_suite.go
* validate bundler txs * nits * lint (cherry picked from commit 339b927)
…157) * fix(auction): Adding extra check on bundler timeouts (#156) * validate bundler txs * nits * lint (cherry picked from commit 339b927) # Conflicts: # tests/app/app.go # tests/integration/block_sdk_suite.go * fix --------- Co-authored-by: David Terpay <35130517+davidterpay@users.noreply.github.com> Co-authored-by: David Terpay <david.terpay@gmail.com>
Overview
This PR enforces that every transaction within a searchers bundle has the same timeout height as the bid transaction. By doing so, we ensure that searcher's bundled transactions are not included by other searchers in subsequent blocks in the case when their bid does not get selected.
Additionally, we limit bidding exclusively for the next block height. This is another mechanism to ensure that searchers do not have their bundles un-bundled.
Testing
Added integration unit tests testing the various different timeout scenarios.