Beginner FriendlyFoundry
100 EXP
View results
Submission Details
Severity: low
Invalid

QA: The event name `SetNetPassword()` should probably be `SetNewPassword()`, as the current name could be misleading/confusing.

Summary

Misleading because it implies setting a net password. But this contract is for setting any password and storing it in the contract. Since set new password is more in line with the intention of the contract, the event name should be adapted accordingly.

Current misleading event name:

event SetNetPassword();

Vulnerability Details

n/a

Impact

n/a

Tools Used

VSC.

Recommendations

-- event SetNetPassword();
++ event SetNewPassword();
Updates

Lead Judging Commences

inallhonesty Lead Judge
almost 2 years ago
inallhonesty Lead Judge almost 2 years ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

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