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

Consider limiting redelegation/ubd entries as spam prevention #3270

Closed
rigelrozanski opened this issue Jan 10, 2019 · 2 comments · Fixed by #3372
Closed

Consider limiting redelegation/ubd entries as spam prevention #3270

rigelrozanski opened this issue Jan 10, 2019 · 2 comments · Fixed by #3372

Comments

@rigelrozanski
Copy link
Contributor

follow up to the implementation of #1402

I think we should consider a basic spam measure to limit the number of Entries in either Redelegation or Unbonding-Delegation to a staking parameter - for starters we could set it at say 50 (aka you can only have up to 50 incomplete unbonding delegations between a delegator-address/validator-address pair - per unbonding period) - thoughts? @cwgoes ?

@cwgoes
Copy link
Contributor

cwgoes commented Jan 10, 2019

That sounds fine; I think we can be way more conservative, e.g. 5-10.

@cwgoes
Copy link
Contributor

cwgoes commented Jan 16, 2019

We should absolutely do this prelaunch, I think we'd open ourselves up to spam otherwise.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants