folkyatina
commented
1 year ago Looks good and even close the issue 5.2 from the old audit that we previously acknowledged.
One thing bothers me:
- INITIAL_TOKEN_HOLDER defined in StETH, but used in Lido.
- StETH is not actually usable without calling
_bootstrapInitialHolder()but nothing in StETH tells you about it.
But I'm not sure what is the best thing to do here. Probably, adding explicit _mintInitialShares() or at least a comment will work.
hasInitializedonresume