-
Notifications
You must be signed in to change notification settings - Fork 444
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
Add isSequencer mapping to SequencerInbox #1424
Conversation
Codecov Report
@@ Coverage Diff @@
## contracts-develop #1424 +/- ##
=====================================================
+ Coverage 48.38% 52.29% +3.90%
=====================================================
Files 245 260 +15
Lines 29528 34120 +4592
Branches 555 555
=====================================================
+ Hits 14287 17842 +3555
- Misses 13196 14200 +1004
- Partials 2045 2078 +33 |
* @notice Updates whether an address is authorized to be a sequencer at the sequencer inbox | ||
* @param addr the address | ||
* @param isSequencer_ if the specified address should be authorized as a sequencer |
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.
I think we should add a @dev here that this is not validated onchain right now.
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.
I added a dev comment describing briefly where the IsSeqeuncer information is used. Please let me know if that's what you had in mind
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.
LGTM
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.
LGTM
Add isSequencer mapping in the SequencerInbox for feed message signing (and possibly redis signing)