-
Notifications
You must be signed in to change notification settings - Fork 18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: Add FMA superchainerc20 standard #102
feat: Add FMA superchainerc20 standard #102
Conversation
There is another possible failure mode of different token addresses between chains. The fix for this would be to have good deterministic deployment tooling around battle tested create2 deployers |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This make sense with the current
https://github.com/ethereum-optimism/design-docs/blob/main/protocol/superchainERC20/superc20Redesign.md
- I would like to know also what happen if people are sending token directly to the
SuperchainERC20Bridge
do we have method to rescue them?
Can this put the bridge in a inconsistent state (like if this is usingSuperTokenERC20.balanceOf(this);
). This probably bit out of the scope but I am interested in it.
Otherwise looks go to me 🚀
Sure, @Ethnical. Currently, there is no plan to add a rescue method yet; for now, we prefer to stick with minimal logic around the contract. Regarding the possibility of an inconsistent state, the bridge doesn’t hold state or care about its token balance, so it shouldn’t be a problem. Here is a reference implementation. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good to me, cannot think of any other real risks
Description
This PR introduces the FMA for the
SuperchainERC20
standard.Additional context
It will allow the use of the
SuperchainERC20
standard without requiring interop functionalities to be enabled at that time.