Skip to content

Commit 3d99748

Browse files
committed
fix: onlyOwner for setClaimerFor
1 parent 315bfb1 commit 3d99748

File tree

3 files changed

+13
-9
lines changed

3 files changed

+13
-9
lines changed

src/ServiceManagerBase.sol

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -167,11 +167,9 @@ abstract contract ServiceManagerBase is ServiceManagerBaseStorage {
167167
/**
168168
* @notice Forwards a call to Eigenlayer's RewardsCoordinator contract to set the address of the entity that can call `processClaim` on behalf of this contract.
169169
* @param claimer The address of the entity that can call `processClaim` on behalf of the earner
170-
* @dev Only callabe by the permissioned rewardsInitiator address
170+
* @dev Only callabe by the owner.
171171
*/
172-
function setClaimerFor(
173-
address claimer
174-
) public virtual onlyRewardsInitiator {
172+
function setClaimerFor(address claimer) public virtual onlyOwner {
175173
_rewardsCoordinator.setClaimerFor(claimer);
176174
}
177175

src/interfaces/IServiceManager.sol

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ interface IServiceManager is IServiceManagerUI {
4444
/**
4545
* @notice Forwards a call to Eigenlayer's RewardsCoordinator contract to set the address of the entity that can call `processClaim` on behalf of this contract.
4646
* @param claimer The address of the entity that can call `processClaim` on behalf of the earner
47-
* @dev Only callabe by the permissioned rewardsInitiator address
47+
* @dev Only callabe by the owner.
4848
*/
4949
function setClaimerFor(address claimer) external;
5050

src/unaudited/ECDSAServiceManagerBase.sol

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -115,10 +115,8 @@ abstract contract ECDSAServiceManagerBase is
115115
}
116116

117117
/// @inheritdoc IServiceManager
118-
function setClaimerFor(
119-
address claimer
120-
) public virtual onlyRewardsInitiator {
121-
IRewardsCoordinator(rewardsCoordinator).setClaimerFor(claimer);
118+
function setClaimerFor(address claimer) external virtual onlyOwner {
119+
_setClaimerFor(claimer);
122120
}
123121

124122
/// @inheritdoc IServiceManagerUI
@@ -263,6 +261,14 @@ abstract contract ECDSAServiceManagerBase is
263261
);
264262
}
265263

264+
/**
265+
* @notice Forwards a call to Eigenlayer's RewardsCoordinator contract to set the address of the entity that can call `processClaim` on behalf of this contract.
266+
* @param claimer The address of the entity that can call `processClaim` on behalf of the earner.
267+
*/
268+
function _setClaimerFor(address claimer) internal virtual {
269+
IRewardsCoordinator(rewardsCoordinator).setClaimerFor(claimer);
270+
}
271+
266272
/**
267273
* @notice Retrieves the addresses of all strategies that are part of the current quorum.
268274
* @dev Fetches the quorum configuration from the ECDSAStakeRegistry and extracts the strategy addresses.

0 commit comments

Comments
 (0)