Skip to content

feat: add fma ban deposits#185

Merged
mds1 merged 7 commits intoethereum-optimism:mainfrom
defi-wonderland:feat/fma-bandeposits
Feb 10, 2025
Merged

feat: add fma ban deposits#185
mds1 merged 7 commits intoethereum-optimism:mainfrom
defi-wonderland:feat/fma-bandeposits

Conversation

@Joxess
Copy link
Copy Markdown
Contributor

@Joxess Joxess commented Jan 10, 2025

Description

This PR introduces the FMA for the deposit context feature, in order to ban ExecutingMessage deposits.

Additional context

Below are the references for this project:

  • Contracts and client updates are documented here.
  • Implementation and tests are found here.

@Inphi
Copy link
Copy Markdown
Contributor

Inphi commented Jan 24, 2025

Are interop deposits still being banned? What happened to the pre-registration idea?

@Joxess
Copy link
Copy Markdown
Contributor Author

Joxess commented Jan 24, 2025

Are interop deposits still being banned? What happened to the pre-registration idea?

@Inphi Yes, currently ExecutingMessage is being banned from deposits, but there is early work in that direction: defi-wonderland/optimism#210

Comment thread security/fma-ban-deposits.md
Comment thread security/fma-ban-deposits.md
Comment thread security/fma-ban-deposits.md Outdated
Comment thread security/fma-ban-deposits.md Outdated
Comment thread security/fma-ban-deposits.md Outdated
Comment thread security/fma-ban-deposits.md Outdated
Comment thread security/fma-ban-deposits.md Outdated
@smartcontracts
Copy link
Copy Markdown
Contributor

Ok, approved

@mds1 mds1 merged commit f76a23a into ethereum-optimism:main Feb 10, 2025
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.

5 participants