celestiaorg / optimism

The Optimism monorepo

Home Page:https://optimism.io

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Hardcoded DA config on NewL2Verifier

upnodedev opened this issue · comments

func NewL2Verifier(t Testing, log log.Logger, l1 derive.L1Fetcher, eng L2API, cfg *rollup.Config) *L2Verifier {
metrics := &testutils.TestDerivationMetrics{}
daCfg, err := rollup.NewDAConfig("http://localhost:26658", "0000e8e5f679bf7116cb", "")
require.NoError(t, err)
pipeline := derive.NewDerivationPipeline(log, cfg, daCfg, l1, eng, metrics)
pipeline.Reset()

We have found that you have hardcoded DA to a local node in NewL2Verifier of l2_verifier.go.

Is this intentionally?

Codes in op-e2e are testcases. So, it's not wrong.