coderabbitai[bot]
commented
2 months ago [!WARNING]
Rate limit exceeded
@crnbarr93 has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 19 minutes and 11 seconds before requesting another review.
How to resolve this issue?
After the wait time has elapsed, a review can be triggered using the `@coderabbitai review` command as a PR comment. Alternatively, push new commits to this PR. We recommend that you space out your commits to avoid hitting the rate limit.How do rate limits work?
CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our [FAQ](https://coderabbit.ai/docs/faq) for further information.Commits
Files that changed from the base of the PR and between fa3952bad0a8968612f87e90fed5f29edae9180f and f57210b5185dc7766178c6145377ee57e83d271e.
Walkthrough
The changes encompass updates to version numbers across multiple Cargo.toml files, reflecting minor increments typically associated with bug fixes or small enhancements. Additionally, a new script, bump-version.sh, has been introduced to automate version number updates. Significant modifications to permissioning logic have been made in the permissioning.rs files, enhancing control flow and permission management. The integration tests have also been updated to dynamically retrieve contract versions from Cargo.toml.
Changes
| Files | Change Summary |
|---|---|
CHANGELOG.md |
Added entry for a fix related to permissioning and security enhancements. |
bump-version.sh |
Introduced a script to automate version number updates in Cargo.toml files. |
contracts/.../Cargo.toml (various) |
Incremented version numbers across multiple packages, indicating minor updates. |
packages/std/src/ado_base/permissioning.rs |
Modified logic in LocalPermission for blacklisting and usage consumption. |
packages/std/src/ado_contract/permissioning.rs |
Updated permission handling logic and tests for action consumption based on permissions. |
tests-integration/Cargo.toml |
Removed commented dependencies and added version specification for the toml crate. |
tests-integration/tests/cw20_staking.rs |
Introduced a function to dynamically retrieve contract version from Cargo.toml. |
Sequence Diagram(s)
sequenceDiagram
participant User
participant Permissioning
participant Contract
User->>Permissioning: Request action
Permissioning->>Permissioning: Check if action is permissioned
alt Action is permissioned
Permissioning->>Contract: Consume permission
else Action is not permissioned
Permissioning->>User: Deny action
end🐰 In the garden, I hop with glee,
Versions updated, oh what a spree!
Scripts to bump, permissions refined,
Bugs fixed swiftly, all intertwined.
A joyful dance, with code so bright,
Hopping along, everything feels right! 🌼✨
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?
crnbarr93
Motivation
These changes include the following:
Implementation
strictparameter when checking an expired blacklistis_context_permissionedwhen origin has permissionTesting
Test cases were added for the provided scenarios
Version Changes
andromeda-std:1.2.2->1.2.3Bumped all ADO contract versionsSummary by CodeRabbit
New Features
Cargo.tomlfor improved test accuracy.Bug Fixes
Documentation
CHANGELOG.mdto reflect recent changes and improvements.Chores