Skip to content
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: Staking Router 1.5 #860

Draft
wants to merge 204 commits into
base: develop
Choose a base branch
from
Draft

feat: Staking Router 1.5 #860

wants to merge 204 commits into from

Conversation

tamtamchik
Copy link
Member

Port of #78

mkurayan and others added 30 commits May 23, 2024 18:08
Depending on the size of the third-phase report, it may be split into multiple transactions.
Distribute reward separately using the permissionless method in each staking module.
…-third-phase-fix

Feat/multi transaction oracle third phase fix
Update contract version and reward distribution state.
new parameter placed at the end of struct to preserve data order compatibility on read
eddort and others added 30 commits July 30, 2024 10:45
Update limit according to latest estimation of gas consumption in CSM module
Update limits to accurate values based on
 - Target max third phase transaction gas cost
 - Max single node operators update gas cost
…ock during module update

Ackee L1: Overflow on type casting.
In the staking router, ensure that the new values for minDepositBlockDistance
and maxDepositsPerBlock are checked for potential overflow during module updates to avoid issues.
Ackee L2: Potential revert on underflow.
In the Staking Router, an unsafe update of the exited validators count could potentially cause
the total exited validators count to exceed the total deposited validators count.

This would break the pre-defined invariant:
activeValidatorsCount = totalDepositedValidators - totalExitedValidators.

This fix ensures the integrity of the active validators count by preventing such an underflow.
Ackee L3: The clearNodeOperatorPenalty returns always false
Ackee I2: Missing event on clearNodeOperatorPenalty
Ackee I3: Typo in node operator registry function name
…ocator

Use steth on optimism locator impl to fetch addresses for SR2 deploy
…limit

Update CL_BALANCE_ORACLES_ERROR_UPPER_BP_LIMIT
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.

7 participants