fix(dtl): sync bug when addresses change

Fixes a bug in the DTL that would break sync when addresses are changed in the AddressManager. Slightly reduces complexity at the same time by removing an additional request.
ethereum-optimism · Aug 17, 2022 · 3dac8f0 · 3dac8f0
1 parent e4ef92a
commit 3dac8f0
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/.changeset/weak-moose-hope.md b/.changeset/weak-moose-hope.md
@@ -0,0 +1,5 @@
+---
+'@eth-optimism/data-transport-layer': patch
+---
+
+Fixes a bug that would break DTL sync when addresses are changed
diff --git a/packages/data-transport-layer/src/services/l1-ingestion/service.ts b/packages/data-transport-layer/src/services/l1-ingestion/service.ts
@@ -374,10 +374,7 @@ export class L1IngestionService extends BaseService<L1IngestionServiceOptions> {
     let l1BlockRangeStart = fromL1Block
     for (const addressSetEvent of addressSetEvents) {
       eventRanges.push({
-        address: await this._getContractAddressAtBlock(
-          contractName,
-          addressSetEvent.blockNumber
-        ),
+        address: addressSetEvent.args._oldAddress,
         fromBlock: l1BlockRangeStart,
         toBlock: addressSetEvent.blockNumber,
       })