Skip to content

Commit

Permalink
fix: setStrategyStatus can be called by strategist.
Browse files Browse the repository at this point in the history
  • Loading branch information
clement-ux committed Dec 30, 2024
1 parent 500b6d3 commit 23a885f
Showing 1 changed file with 9 additions and 1 deletion.
10 changes: 9 additions & 1 deletion contracts/contracts/harvest/OETHHarvesterSimple.sol
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,14 @@ contract OETHHarvesterSimple is Governable {
event StrategistSet(address strategist);
event StrategyStatusSet(address strategy, bool status);

////////////////////////////////////////////////////
/// --- MODIFIERS
////////////////////////////////////////////////////
modifier onlyStrategist() {
require(msg.sender == strategist || isGovernor(), "Not strategist");
_;
}

////////////////////////////////////////////////////
/// --- CONSTRUCTOR
////////////////////////////////////////////////////
Expand Down Expand Up @@ -67,7 +75,7 @@ contract OETHHarvesterSimple is Governable {
////////////////////////////////////////////////////
function setStrategyStatus(address _strategy, bool _status)
external
onlyGovernor
onlyStrategist
{
isAuthorized[_strategy] = _status;
emit StrategyStatusSet(_strategy, _status);
Expand Down

0 comments on commit 23a885f

Please sign in to comment.