Commit b1cec1d5 authored by smartcontracts's avatar smartcontracts Committed by GitHub

fix(dtl): sync bug when addresses change (#3241)

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.
Co-authored-by: default avatarMatthew Slipper <me@matthewslipper.com>
Co-authored-by: default avatarMark Tyneway <mark.tyneway@gmail.com>
parent 0bc1be45
---
'@eth-optimism/data-transport-layer': patch
---
Fixes a bug that would break DTL sync when addresses are changed
...@@ -374,10 +374,7 @@ export class L1IngestionService extends BaseService<L1IngestionServiceOptions> { ...@@ -374,10 +374,7 @@ export class L1IngestionService extends BaseService<L1IngestionServiceOptions> {
let l1BlockRangeStart = fromL1Block let l1BlockRangeStart = fromL1Block
for (const addressSetEvent of addressSetEvents) { for (const addressSetEvent of addressSetEvents) {
eventRanges.push({ eventRanges.push({
address: await this._getContractAddressAtBlock( address: addressSetEvent.args._oldAddress,
contractName,
addressSetEvent.blockNumber
),
fromBlock: l1BlockRangeStart, fromBlock: l1BlockRangeStart,
toBlock: addressSetEvent.blockNumber, toBlock: addressSetEvent.blockNumber,
}) })
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment