• 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
..
actor-tests Loading commit data...
common-ts Loading commit data...
contracts Loading commit data...
contracts-bedrock Loading commit data...
contracts-governance Loading commit data...
contracts-periphery Loading commit data...
core-utils Loading commit data...
data-transport-layer Loading commit data...
drippie-mon Loading commit data...
fault-detector Loading commit data...
hardhat-deploy-config Loading commit data...
integration-tests-bedrock Loading commit data...
message-relayer Loading commit data...
replica-healthcheck Loading commit data...
sdk Loading commit data...