search

manifoldfinance / mevETH2

mevETH LST Protocol - Repo has migrated see link
https://github.com/MEV-Protocol/meveth
27 stars 2 forks source link

audit: secure redeemCream function with min deposit, pause check and c… #183

Closed sandybradley closed 1 year ago

sandybradley commented 1 year ago

…orrect cream burn

closes https://github.com/manifoldfinance/mevETH2/issues/193 closes https://github.com/manifoldfinance/mevETH2/issues/191 closes https://github.com/manifoldfinance/mevETH2/issues/189

ControlCplusControlV commented 1 year ago

Given the severity of the issue at play here going to request we have some fork tests to ensure the fix works as expected, also while these do make sense here make sure to usually keep 1 issue per pr

sandybradley commented 1 year ago

@ControlCplusControlV fork test added