Raisebox Faucet

First Flight #50
Beginner FriendlySolidity
100 EXP
View results
Submission Details
Impact: low
Likelihood: low
Invalid

Low vulnerabilities


  • mintFaucetTokens: Current implementation only allows minting to address(this). It uses a revert for any other address. to ! = address(this) is restrictive and may break future integrations. Suggest removing the to parameter restriction if not necessary.

  • adjustDailyClaimLimit: Owner can increase or decrease the daily claim limit. Suggest wrapping in unchecked dailyClaimLimit += by bcz using Solidity >=0.8 and checks already ensured. Emit event DailyClaimLimitAdjusted(address indexed owner, uint256 newLimit, bool increased) to track changes off-chain.

  • Remove getClaimer() bcz this function is unnecesary and to get all claimers we could use event Claim and offChain to get all claimers.

  • In DeployRaiseBoxFaucet.s.sol typo error //SPDX-Lincense-Identifier: MIT replace to //SPDX-License-Identifier: MIT


Updates

Lead Judging Commences

inallhonesty Lead Judge 7 days ago
Submission Judgement Published
Invalidated
Reason: Lack of quality

Support

FAQs

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