Skip to content

Disallow L1 Message transactions in pool #202

Open
@greged93

Description

@greged93

Describe the bug

The current pool validator implementation does not disallow for L1 messages to be sent via the transactions pool.

pub fn validate_one(
&self,
origin: TransactionOrigin,
transaction: Tx,
) -> TransactionValidationOutcome<Tx> {
if transaction.is_eip4844() {
return TransactionValidationOutcome::Invalid(
transaction,
InvalidTransactionError::TxTypeNotSupported.into(),
)
}
let outcome = self.inner.validate_one(origin, transaction);

The transactions will however be rejected during block building but we could go ahead and directly reject them at mempool level.

if tx.is_eip4844() || tx.is_l1_message() {

Steps to reproduce

NA

Node logs


Platform(s)

No response

Container Type

Not running in a container

What version/commit are you on?

NA

What database version are you on?

NA

Which chain / network are you on?

scroll-mainnet

What type of node are you running?

Archive (default)

What prune config do you use, if any?

No response

If you've built Reth from source, provide the full command you used

No response

Code of Conduct

  • I agree to follow the Code of Conduct

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    No type

    Projects

    Status

    No status

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions