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

QA: The `SetNetPassword()` event contains no parameters, but ideally should contain at least 1 to return some useful info.

Summary

The main purpose of events in Solidity is to provide a way for smart contracts to communicate and log important information to the Ethereum blockchain, allowing external applications and users to track and react to specific contract actions and state changes.

So with the above in mind, the current event with zero parameters wont really provide any useful info (to the user).

The event as its currently implemented without any parameters:

event SetNetPassword();

Vulnerability Details

n/a

Impact

n/a

Tools Used

VSC.

Recommendations

event SetNetPassword(address indexed owner, string password);

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.