scroll-tech / go-ethereum

Scroll's fork of the official Go implementation of the Ethereum protocol
GNU Lesser General Public License v3.0
494 stars 276 forks source link

fix: skip ccc.ErrUnknown txns immediately #1027

Closed omerfirmak closed 1 month ago

omerfirmak commented 1 month ago

1. Purpose or design rationale of this PR

otherwise they might keep triggering reorgs until they happen to be the txn at index 0

2. PR title

Your PR title must follow conventional commits (as we are doing squash merge for each PR), so it must start with one of the following types:

3. Deployment tag versioning

Has the version in params/version.go been updated?

4. Breaking change label

Does this PR have the breaking-change label?

omerfirmak commented 1 month ago

nice catch.

props to @Thegaram for raising this