Relying on block.timestamp
and msg.sender
to generate a random number in not good because the number is not completely random and can be predicted
In openBox
function it calculate the randomValue
as result of hashing block.timestamp
and msg.sender
% 100 which a weak source of randomness .
User can get always the highest reward when he open a box
Manual review
Use Source of randomness 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.