use CONTEXT instead of ERC2771 CONTEXT & msg.sender is not consistent in the codebase
Summary
Vulnerability Details
Throughout the codebase, the code uses Context for _msgSender()
The implementation chosen will resolve each _msgSender() to msg.sender which is inconsistent with the goal of
allowing MetaTX.
The code is using OpenZeppelin's Context contract which is intended to allow meta-transactions. It works by using doing a call to _msgSender() instead of querying msg.sender directly, because the method allows those special transactions.
Impact
_msgSender() to msg.sender which is inconsistent with the goal of allowing MetaTX.
Tools Used
Manual Review
Recommendations
Replace the import of Context with ERC2771Contex and as reported by the bot Use _msgsender everywhere instead of msgsender
Prize pool breakdown
Live
The contest is live. Earn rewards by submitting a finding.
Appeals
This is your time to appeal against judgements on your submissions.
Appeals Review
Appeals are being carefully reviewed by our judges.