The events CheckedOnce
and CheckedTwice
in the SantasList
contract are missing the indexed
keyword for their fields.
The events should utilize the indexed
keyword for fields that may be useful for off-chain tools parsing events. Indexing these fields helps in quick accessibility of the event data. In this case, the events CheckedOnce
and CheckedTwice
should consider using the indexed
keyword for the fields person
and status
to improve the efficiency of off-chain tools.
The absence of the indexed
keyword may not affect the on-chain functionality of the contract, but it could impact the efficiency of off-chain tools that parse these events. Indexing relevant fields helps in quicker data retrieval for certain queries.
Manual review
Consider adding the indexed
keyword to the relevant fields in the CheckedOnce
and CheckedTwice
events. For example:
This change can enhance the efficiency of off-chain tools that parse these events without significantly impacting on-chain gas costs unless the number of indexed fields exceeds three.
The contest is live. Earn rewards by submitting a finding.
This is your time to appeal against judgements on your submissions.
Appeals are being carefully reviewed by our judges.