fix: use empty access list if eth_createAccessList returns error (#1799)

Co-authored-by: Péter Garamvölgyi <peter@scroll.io>

Author: georgehao

common/version/version.go

@@ -5,7 +5,7 @@ import (
 	"runtime/debug"
 )
 
-var tag = "v4.7.12"
+var tag = "v4.7.13"
 
 var commit = func() string {
 	if info, ok := debug.ReadBuildInfo(); ok {

rollup/internal/controller/sender/estimategas.go

@@ -1,7 +1,6 @@
 package sender
 
 import (
-	"errors"
 	"fmt"
 	"math/big"
 
@@ -134,11 +133,15 @@ func (s *Sender) estimateGasLimit(to *common.Address, data []byte, sidecar *type
 	accessList, gasLimitWithAccessList, errStr, rpcErr := s.gethClient.CreateAccessList(s.ctx, msg)
 	if rpcErr != nil {
 		log.Error("CreateAccessList RPC error", "error", rpcErr)
-		return gasLimitWithoutAccessList, nil, rpcErr
+		// We ignore errors from eth_createAccessList and proceed
+		// with sending the transaction without an access list.
+		return gasLimitWithoutAccessList, nil, nil
 	}
 	if errStr != "" {
+		// We ignore errors from eth_createAccessList and proceed
+		// with sending the transaction without an access list.
 		log.Error("CreateAccessList reported error", "error", errStr)
-		return gasLimitWithoutAccessList, nil, errors.New(errStr)
+		return gasLimitWithoutAccessList, nil, nil
 	}
 
 	// Fine-tune accessList because 'to' address is automatically included in the access list by the Ethereum protocol: https://github.com/ethereum/go-ethereum/blob/v1.13.10/core/state/statedb.go#L1322