Description:Description\
The functions with onlyGovernor modifier that change critical parameters should emit events.
Sensitive setter functions in smart contracts often alter critical state variables. Without events emitted in these functions, external observers or dApps cannot easily track or react to these state changes. Missing events can obscure contract activity, hampering transparency and making integration more challenging. To resolve this, incorporate appropriate event emissions within these functions. Events offer an efficient way to log crucial changes, aiding in real-time tracking and post-transaction verification.
clesaege
commented
3 weeks ago
Github username: @dinkras Twitter username: dinkras Submission hash (on-chain): 0xe4b5ae49b8e24d1563067ded8273570ce6627b9b77223d190038f70b5f376719 Severity: low
Description: Description\ The functions with
onlyGovernormodifier that change critical parameters should emit events.Sensitive setter functions in smart contracts often alter critical state variables. Without events emitted in these functions, external observers or dApps cannot easily track or react to these state changes. Missing events can obscure contract activity, hampering transparency and making integration more challenging. To resolve this, incorporate appropriate event emissions within these functions. Events offer an efficient way to log crucial changes, aiding in real-time tracking and post-transaction verification.
Those the functions that lack event emissions.
This also creates inconsistencies with the other contracts that have
onlyGovernormodifier functions and emit events(ProofOfHumanity.sol)Recommendation
Add events to all
onlyGovernorfunctions that change critical parameters.