Skip to content

Conversation

@krebernisak
Copy link
Contributor

  • Add L1 & L2 contracts required for the Optimism token bridge (solc v0.7)
  • Make L1/L2 Gateways an ERC677Receiver so transferAndCall can be used to avoid approving tokens before deposit/withdraw
  • Add Optimism integration test
    • L1 -> L2 -> L1 deposit-wihdraw integration test
    • L2 withdrawals safety check integration test (blocking contract withdrawals but allowing EOA contracts to withdraw)
  • Expose methods like deployGateways to deploy the L1<->L2 bridge that consumers might find useful.
  • Add proxy deployment for the bridge contracts (L1 & L2 gateways)

TODO:

  • Define genesis ProxyEOA.sol codehash in OVM_EOACodeHashSet.sol
  • Document the reasoning behind "unsafe" bridge operations
  • Document upgradebility and bridge extension scenarios
    • Add deployment scenario (LinkTokenChild first, bridge second) integration test
  • Get feedback
  • Polish & finalize contracts
  • Audit & approve

@krebernisak krebernisak force-pushed the feature/oe-token branch 2 times, most recently from e9afcef to 9464717 Compare April 27, 2021 18:01
@krebernisak krebernisak changed the title Optimism L2 bridge Optimism L2 - bridge Apr 27, 2021
@krebernisak krebernisak force-pushed the feature/oe-bridge branch 2 times, most recently from 7bf5250 to 78c1c15 Compare April 28, 2021 12:15
@krebernisak krebernisak force-pushed the feature/oe-bridge branch from 43e7da2 to 3ffdba8 Compare May 6, 2021 11:43
@krebernisak krebernisak force-pushed the feature/oe-bridge branch from 3ffdba8 to 3a68fba Compare May 7, 2021 17:41
@krebernisak krebernisak force-pushed the feature/oe-bridge branch from 3a68fba to 795a8e9 Compare May 9, 2021 20:52
@krebernisak krebernisak force-pushed the feature/oe-bridge branch 3 times, most recently from 9f621b3 to cd3fc26 Compare May 11, 2021 20:52
@krebernisak krebernisak force-pushed the feature/oe-bridge branch 3 times, most recently from 4e4524f to 894b25b Compare May 13, 2021 10:17
- Add Optimism L2 bridge contracts
- Add Optimism specific OVM_EOACodeHashSet contract
- Add ERC677Receiver functionality to L1/L2 gateways
- Add L1-L2 deposit-withdraw bridge integration tests
- Force compile OZ TransparentUpgradeableProxy.sol & ProxyAdmin.sol
@krebernisak krebernisak changed the base branch from feature/oe-token to master June 9, 2021 10:41
@krebernisak krebernisak changed the base branch from master to feature/oe-token June 9, 2021 10:41
@krebernisak
Copy link
Contributor Author

Closing this in favor of using the new Optimism standard bridge v2.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant