Skip to content
This repository has been archived by the owner on Jun 6, 2023. It is now read-only.

Verifreg.AddVerifier: params.Allowance should be greater than or equal to MinVerifiedDealSize #724

Closed
wadealexc opened this issue Jul 20, 2020 · 0 comments · Fixed by #758
Closed

Verifreg.AddVerifier: params.Allowance should be greater than or equal to MinVerifiedDealSize #724

wadealexc opened this issue Jul 20, 2020 · 0 comments · Fixed by #758
Assignees
@aarshkshah1992
Labels
P2 Medium priority, beneficial for network functionality and growth

Comments

@wadealexc
Copy link

Currently, Verifiers can be added whose allowance doesn't even allow them to add a VerifiedClient. By enforcing that params.Allowance >= MinVerifiedDealSize, this should never happen.
@aarshkshah1992 aarshkshah1992 self-assigned this Jul 22, 2020
@aarshkshah1992 aarshkshah1992 mentioned this issue Jul 22, 2020
Merged
@anorth anorth added the P2 Medium priority, beneficial for network functionality and growth label Jul 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@aarshkshah1992 aarshkshah1992

Labels
P2 Medium priority, beneficial for network functionality and growth
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

VerifReg: Audit Fixes filecoin-project/specs-actors
3 participants
@anorth @aarshkshah1992 @wadealexc