• Mark Tyneway's avatar
    contracts-bedrock: L2CrossDomainMessenger legibility (#3266) · fec22bfe
    Mark Tyneway authored
    The `initialize()` function in the L2CrossDomainMessenger
    sets some blocked system addresses. It was using `address(this)`
    but in reality the contract is a predeploy at a specific address.
    It should not be deployed, the `initialize` function will never
    run. For the unit tests, the contracts are placed at their correct
    addresses. This makes the testing environment as close as possible
    to what is experienced on a live network.
    
    This PR hardcodes in the L2 cross domain messenger constant
    from the predeploys library instead of using `address(this)`.
    In practice this value is set using the chain ops tooling,
    so this makes it more clear as to what the value should be.
    Co-authored-by: default avatarmergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
    fec22bfe
Name
Last commit
Last update
..
addressmanager.go Loading commit data...
gaspriceoracle.go Loading commit data...
gaspriceoracle_deployed.go Loading commit data...
l1block.go Loading commit data...
l1block_deployed.go Loading commit data...
l2crossdomainmessenger.go Loading commit data...
l2outputoracle.go Loading commit data...
l2standardbridge.go Loading commit data...
l2tol1messagepasser.go Loading commit data...
l2tol1messagepasser_deployed.go Loading commit data...
optimismportal.go Loading commit data...
optimismportal_deployed.go Loading commit data...
sequencerfeevault.go Loading commit data...