makerdao / optimism-dai-bridge Goto Github PK

For example , in contracts/L2/L2DAITokenBridge.sol

there is

_initiateWithdrawal(msg.sender, msg.sender, _amount, _l1Gas, _data); 

which should be

_initiateWithdrawal(msg.sender, msg.value ,  _amount, _l1Gas, _data);  

Since the second argument is amount , it should be msg.value

In the L2 Dai implementation, governance has the ability to burn Dai in any address.

This enables Dai censorship of a user's wallet by governance, which means that we can't make the claim that L2 Dai is censorship-resistant as it is on L1.

This needs a broad team, and potentially MKR holder, discussion.

Hi there!

Where can I find the target contracts used in delegatecall, for instance, here:
https://github.com/makerdao/optimism-dai-bridge/blob/master/contracts/l2/L2GovernanceRelay.sol#L45

Thanks!