-
Notifications
You must be signed in to change notification settings - Fork 124
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
Update default rate limit params for USDC #1056
Conversation
Important Auto Review SkippedDraft detected. Please check the settings in the CodeRabbit UI or the To trigger a single review, invoke the WalkthroughThe recent updates involve adjusting the baseline limits for rate-limiting within a blockchain protocol. Specifically, the changes increase the minimum baseline values for rate-limiters, enhancing the thresholds for 1-hour and 1-day limits from 100k and 1m USDC to 1m and 10m USDC respectively. This adjustment reflects a significant increase in the capacity for transaction rate-limiting, aiming to accommodate larger volumes of transactions. Changes
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Review Status
Actionable comments generated: 0
Configuration used: CodeRabbit UI
Files ignored due to path filters (2)
protocol/app/testdata/default_genesis_state.json
is excluded by:!**/*.json
protocol/scripts/genesis/sample_pregenesis.json
is excluded by:!**/*.json
Files selected for processing (3)
- protocol/testutil/constants/genesis.go (1 hunks)
- protocol/x/ratelimit/types/params.go (1 hunks)
- protocol/x/ratelimit/types/params_test.go (1 hunks)
Additional comments: 6
protocol/x/ratelimit/types/params_test.go (2)
- 19-19: The
BaselineMinimum
value for the 1-hour limiter has been updated to1_000_000_000_000
. Confirm this aligns with the intended rate limit adjustments.- 24-24: The
BaselineMinimum
value for the 1-day limiter has been updated to10_000_000_000_000
. Ensure this change matches the protocol's updated rate limiting strategy.protocol/x/ratelimit/types/params.go (2)
- 16-16: The
BigBaselineMinimum1Hr
value has been updated to1_000_000
. Verify the calculation correctly accounts for the USDC denomination exponent.- 22-22: The
BigBaselineMinimum1Day
value has been updated to10_000_000
. Confirm the calculation is accurate, considering the USDC denomination exponent.protocol/testutil/constants/genesis.go (2)
- 1369-1369: The
baseline_minimum
value for the 1-hour limiter has been updated from100000000000
to1000000000000
. Ensure this tenfold increase aligns with the intended rate limit adjustments for USDC transactions.- 1374-1374: The
baseline_minimum
value for the 1-day limiter has been updated from1000000000000
to10000000000000
. Confirm this tenfold increase is consistent with the desired changes for managing USDC transaction volumes over a 1-day period.
@Mergifyio backport release/protocol/v4.x |
✅ Backports have been created
|
(cherry picked from commit d3273b1)
Changelist
Update rate limit params per discussion with research (thread):
Test Plan
Unit tests
Author/Reviewer Checklist
state-breaking
label.indexer-postgres-breaking
label.PrepareProposal
orProcessProposal
, manually add the labelproposal-breaking
.feature:[feature-name]
.backport/[branch-name]
.refactor
,chore
,bug
.