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

Overall and throughput-based IBC rate limits #2552

Closed
Tracked by #2531
cwgoes opened this issue Feb 7, 2024 · 0 comments · Fixed by #2698
Closed
Tracked by #2531

Overall and throughput-based IBC rate limits #2552

cwgoes opened this issue Feb 7, 2024 · 0 comments · Fixed by #2698
Assignees
Labels
IBC pre-mainnet Must happen before mainnet. security

Comments

@cwgoes
Copy link
Contributor

cwgoes commented Feb 7, 2024

We should have IBC rate limits for deposits into and withdrawals out of Namada, as follows:

  • All asset denominations have a deposit limit (by default: 0, meaning that no deposits are allowed). No deposit which would increase the supply on Namada to more than the deposit limit is allowed.
  • All asset denominations have a per-epoch net throughput limit (by default: 0, meaning that no throughput is allowed). This net throughput limit tracks the absolute value of deposits - withdrawals during the epoch. Deposits or withdrawals which would cause the throughput to exceed the limit are not allowed. Tracking net throughput is necessary to prevent DoS.

These limits serve to limit risk in the case of state machine bugs on Namada. Context.

@cwgoes cwgoes mentioned this issue Feb 7, 2024
13 tasks
@cwgoes cwgoes added the pre-mainnet Must happen before mainnet. label Feb 7, 2024
@yito88 yito88 mentioned this issue Feb 22, 2024
2 tasks
@brentstone brentstone added this to the Phase 1: mainnet genesis milestone Apr 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
IBC pre-mainnet Must happen before mainnet. security
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants