-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
op-chain-ops: Ignore messages from sources other than the L2XDM #4861
Conversation
|
fe588fb
to
be7524e
Compare
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## develop #4861 +/- ##
===========================================
- Coverage 39.79% 34.77% -5.03%
===========================================
Files 321 181 -140
Lines 19212 15123 -4089
Branches 761 0 -761
===========================================
- Hits 7646 5259 -2387
+ Misses 10996 9330 -1666
+ Partials 570 534 -36
Flags with carried forward coverage won't be shown. Click here to find out more. |
This PR has been added to the merge queue, and will be merged soon. |
This PR is next in line to be merged, and will be merged as soon as checks pass. |
The
relayMessage
function on the message passer is public. To guard against spoofed withdrawals, we need to check the message's source to ensure that only messages from the L2XDM are included in the withdrawals migration.Fixes CLI-3337
Fixes CLI-3331