QA Report #25
Labels
bug
Something isn't working
QA (Quality Assurance)
Assets are not at risk. State handling, function incorrect as to spec, issues with clarity, syntax
sponsor acknowledged
Technically the issue is correct, but we're not going to resolve it for XYZ reasons
valid
[L-01] Remove TODOs
Two to-do's are still visible within the contract.
https://github.com/Badger-Finance/vested-aura/blob/d504684e4f9b56660a9e6c6dfb839dcebac3c174/contracts/MyStrategy.sol#L284
https://github.com/Badger-Finance/vested-aura/blob/d504684e4f9b56660a9e6c6dfb839dcebac3c174/contracts/MyStrategy.sol#L422
[L-02] Lack of event emissions on setters and state-changing functions
The only function that emits an event is the
_sendTokenToBribesProcessor()
function. All others lack event emissions. Most importantly, setters such assetWithdrawalSafetyCheck()
,setProcessLocksOnReinvest()
, andsetBribesProcessor()
should emit emissions to produce an accurate history of their configuration.The text was updated successfully, but these errors were encountered: