Skip to content

Commit 9e34108

Browse files
committed
chore(protocol-contracts): add comment on transfer beneficiary
1 parent 0ecf976 commit 9e34108

1 file changed

Lines changed: 4 additions & 0 deletions

File tree

protocol-contracts/staking/contracts/OperatorRewarder.sol

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -292,6 +292,10 @@ contract OperatorRewarder is Ownable {
292292
/**
293293
* @notice Transfers the beneficiary address.
294294
* @param newBeneficiary The new beneficiary address.
295+
* @dev Transferring the beneficiary address does not trigger a claim of unclaimed fees for the
296+
* old beneficiary on purpose. This is to avoid losing unclaimed fees in case a beneficiary loses
297+
* access to their private key. It is acceptable as the owner (who can set the beneficiary) is
298+
* expected to be a governance DAO.
295299
*/
296300
function _transferBeneficiary(address newBeneficiary) internal virtual {
297301
require(newBeneficiary != address(0), InvalidBeneficiary(address(0)));

0 commit comments

Comments
 (0)