Hashing msg.sender and block.timestamp together creates a predictable find number. A predictable number is not a good random number. Malicious users can manipulate these values or know them ahead of time to choose the highest value in the mystery box.
Any user can influence the selection of the rewards getting the most valuable reward in the mystery box.
Validators can know ahaed of time the block.timestamp and use that to predict when/how to participate.
Users can mine/manipulate their msg.sender value to result in their address being used to generate the winner.
Users can revert their openBox transaction if they don't like the reward.
Manual code review and slither
Consider using a cryptographically provable random number generator such as Chainlink VRF
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.