Mystery Box

First Flight #25
Beginner FriendlyFoundry
100 EXP
View results
Submission Details
Severity: low
Invalid

Custom error instead of require statement with string message

Summary

The compiler version of MysteryBox.sol is greater than 0.8.0 therefore custom errors are available. Using custom errors will save us more gas than using require statement with string message.

Tools Used

Manual review

Recommendations

Use custom errors instead or require statements with string message.

Example:

+ error Unauthorized();
- require(msg.sender == owner, "Only owner can set price");
+ if(msg.sender != owner){
+ revert Unauthorized();
+ }
boxPrice = _price;
Updates

Appeal created

inallhonesty Lead Judge about 1 year ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

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

Give us feedback!