hats-finance / Metrom-0xfdfc6d4ac5807d7460da20a3a1c0c84ef2b9c5a2

Smart contracts for the Metrom project.
GNU General Public License v3.0
0 stars 0 forks source link

No way to recover the unclaimed rewards. #40

Open hats-bug-reporter[bot] opened 6 months ago

hats-bug-reporter[bot] commented 6 months ago

Github username: -- Twitter username: @flyingBird_246 Submission hash (on-chain): 0xba1576f3560a185af3575e0e4e6ae3b9d797227d236bc0d127927d88a5b79efd Severity: medium

Description: Description\ The function recoverRewards() recovers the unassigned rewards so that the owner of the campaign doesn't lose them. But, the rewards that were assigned to users who didn't claim them for a long time will stuck in the contract forever. Which is a loss for both the user and the owner.

Attachments

  1. Proof of Concept (PoC) File

  2. Revised Code File (Optional) In the attached file I've added a recoverUnclaimedRewards() to mitigate this issue

Files:

luzzif commented 6 months ago

This is actually the intended functionality as to not force the user to do anything and allow the maximum amount of freedom in handling rewards claiming.