Feature request: discard bundles containing reverting transactions

[deci1]

As of now bundles containing reverting transactions can get included in blocks; this could be desired in some situations, but could also expose users to a wide range of risks. My proposal is to add a flag in the sendBundle API that, when set to true, discards bundles containing reverting transactions.

Issue: flashbots/mev-geth#34

[jparyani]

Alternate proposal: in sendBundle, accept a list of txs that are allowed to revert. Reject bundles with a reverting tx not in this list.

[thegostep]

v0.2 proposal

In v0.2, bundles with reverting transactions will no longer be included in blocks, unless explicitly allowed. This change will help avoid unintended transactions from landing on chain leading to less chain bloating.

A new txsAllowingRevert RPC parameter will be included in eth_sendBundle, consisting in an array of transaction hashes. Any reverting transaction not included in this array will lead to the rejection of the entire bundle containing it.