Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Require proof for freeSlot #143

Open
Tracked by #46
emizzle opened this issue Aug 6, 2024 · 0 comments · May be fixed by #191
Open
Tracked by #46

Require proof for freeSlot #143

emizzle opened this issue Aug 6, 2024 · 0 comments · May be fixed by #191
Labels
Marketplace See https://miro.com/app/board/uXjVNZ03E-c=/ for details

Comments

@emizzle
Copy link
Collaborator

emizzle commented Aug 6, 2024

Given that there is leniency in allowed proofs missed before being slashed, slashCriterion, there should be a final proof submitted from SPs at the end of the contract duration before they are paid out?

Rationale

SPs could delete data for the last x periods of the contract, without any penalty at all, where x is: (slashCriterion-1) * proofProbability. Considering that each period could be 24 hours, there is a potential for SPs to be able to delete their contractually obliged data for quite a few days without penalty.

There are also a few incentive structures in place that rely on the idea that SPs will only get paid out after the contract successfully finishes, so adding a single additional proof requirement at the end would only reinforce the idea that data must be stored until the end to get paid out.

Implementation

Add an optional parameter, proof, to freeSlot. On successfully completed requests, SPs must provide a valid proof when calling freeSlot before payout is allowed.

@AuHau AuHau added the Marketplace See https://miro.com/app/board/uXjVNZ03E-c=/ for details label Aug 13, 2024
@emizzle emizzle linked a pull request Oct 8, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Marketplace See https://miro.com/app/board/uXjVNZ03E-c=/ for details
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants