Commit 8ca1e008 authored by Kelvin Fichter's avatar Kelvin Fichter

feat: remove onlyRelayer modifier

parent b70ee70c
...@@ -78,26 +78,6 @@ contract L1CrossDomainMessenger is ...@@ -78,26 +78,6 @@ contract L1CrossDomainMessenger is
{} {}
/**********************
* Function Modifiers *
**********************/
/**
* Modifier to enforce that, if configured, only the OVM_L2MessageRelayer contract may
* successfully call a method.
*/
modifier onlyRelayer() {
address relayer = resolve("OVM_L2MessageRelayer");
if (relayer != address(0)) {
require(
msg.sender == relayer,
"Only OVM_L2MessageRelayer can relay L2-to-L1 messages."
);
}
_;
}
/******************** /********************
* Public Functions * * Public Functions *
********************/ ********************/
...@@ -221,7 +201,6 @@ contract L1CrossDomainMessenger is ...@@ -221,7 +201,6 @@ contract L1CrossDomainMessenger is
) )
public public
nonReentrant nonReentrant
onlyRelayer
whenNotPaused whenNotPaused
{ {
bytes memory xDomainCalldata = Lib_CrossDomainUtils.encodeXDomainCalldata( bytes memory xDomainCalldata = Lib_CrossDomainUtils.encodeXDomainCalldata(
......
...@@ -521,21 +521,5 @@ describe('L1CrossDomainMessenger', () => { ...@@ -521,21 +521,5 @@ describe('L1CrossDomainMessenger', () => {
).to.not.be.reverted ).to.not.be.reverted
}) })
}) })
describe('onlyRelayer', () => {
it('when the OVM_L2MessageRelayer address is set, should revert if called by a different account', async () => {
// set to a random NON-ZERO address
await AddressManager.setAddress(
'OVM_L2MessageRelayer',
'0x1234123412341234123412341234123412341234'
)
await expect(
L1CrossDomainMessenger.relayMessage(target, sender, message, 0, proof)
).to.be.revertedWith(
'Only OVM_L2MessageRelayer can relay L2-to-L1 messages.'
)
})
})
}) })
}) })
...@@ -154,20 +154,6 @@ export class MessageRelayerService extends BaseService<MessageRelayerOptions> { ...@@ -154,20 +154,6 @@ export class MessageRelayerService extends BaseService<MessageRelayerOptions> {
await sleep(this.options.pollingInterval) await sleep(this.options.pollingInterval)
try { try {
// Check that the correct address is set in the address manager
const relayer = await this.state.Lib_AddressManager.getAddress(
'OVM_L2MessageRelayer'
)
// If it is address(0), then message relaying is not authenticated
if (relayer !== ethers.constants.AddressZero) {
const address = await this.options.l1Wallet.getAddress()
if (relayer !== address) {
throw new Error(
`OVM_L2MessageRelayer (${relayer}) is not set to message-passer EOA ${address}`
)
}
}
this.logger.info('Checking for newly finalized transactions...') this.logger.info('Checking for newly finalized transactions...')
if ( if (
!(await this._isTransactionFinalized( !(await this._isTransactionFinalized(
......
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