Snowman Merkle Airdrop

First Flight #42
Beginner FriendlyFoundrySolidityNFT
100 EXP
Submission Details
Impact: low
Likelihood: low
Invalid

### [L-2] The ```FeeCollected``` Event is Declared with zero parameters But Not Emitted in ```Snow::collectFee``` Function.

Author Revealed upon completion

[L-2] The FeeCollected Event is Declared with zero parameters But Not Emitted in Snow::collectFee Function.

Description

The FeeCollected Event is Declared with zero parameters But Not Emitted in Snow::collectFee Function.
this omission of event makes difficult for frontend to trace.

Impact:

1.Difficult to know that how much fee collector has collected or not.

2.Accessing off-chain data becomes hard.

Recommended Mitigation

+event FeeCollected(uint wethamount,uint ethamount);
function collectFee() external onlyCollector {
uint256 collection = i_weth.balanceOf(address(this));
i_weth.transfer(s_collector, collection);
(bool collected, ) = payable(s_collector).call{
value: address(this).balance
}("");
require(collected, "Fee collection failed!!!");
+emit FeeCollected(collection,address(this).balance);
}
Updates

Lead Judging Commences

yeahchibyke Lead Judge 1 day ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

Can't find an answer? Chat with us on Discord, Twitter or Linkedin.