Beginner FriendlyFoundryBridge
100 EXP
View results
Submission Details
Severity: low
Valid

Missing event emition

Summary

Missing successful executed transaction event emition in L1BossBridge::sendToL1()
Missing successful withdrawal event emition in L1BossBridge::withdrawTokensToL1()

Vulnerability Details

L1BossBridge::sendToL1() does not contain an event that is fired when a transaction in successfully executed.
L1BossBridge::withdrawTokensToL1() does not contain an event that is fired when a funds are withdrawn.

Impact

This violates the best practices and disallows to tract the states changes off-chain. This results in lack of transparency and off-chain monitoring capability.

Tools Used

Manual review

Recommendations

Add events and emit them as funds are withdrawn or a transaction in successfully executed.
Otherwise rationalize/document why it isn’t necessary and remove.

Updates

Lead Judging Commences

0xnevi Lead Judge almost 2 years ago
Submission Judgement Published
Validated
Assigned finding tags:

missing events emission

Support

FAQs

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