Closed leonz789 closed 3 months ago
The Lint Code Base error report is for simulation
, could be ignored
[!WARNING]
Review failed
The pull request is closed.
This update primarily enhances the Cosmos SDK's transaction processing by improving signature verification, gas consumption, and sequence handling. Notable changes include switching from the staking to the dogfood module in the context of Exocore validators, updating package versions, and refining the validation mechanics for Oracle price transactions. These changes are aimed at boosting efficiency and robustness in transaction verification processes.
Files | Change Summaries |
---|---|
app/ante/cosmos/sigverify.go |
Introduced signature verification functionality, gas consumption logic, and simulation setup for signatures. |
app/ante/handler_options.go |
Switched decorators to use cosmosante package for signature handling in newCosmosAnteHandler . |
go.mod |
Updated various package versions including cosmossdk.io/errors , cosmossdk.io/math , github.com/cometbft/cometbft , github.com/cosmos/cosmos-sdk , and others. |
local_node.sh |
Added a sed command to replace enable = false with enable = true in a specific file. |
testutil/keeper/oracle.go |
Replaced stakingKeeper.Keeper{} with dogfoodkeeper.Keeper{} in the OracleKeeper function. |
x/oracle/keeper/common/expected_keepers.go |
Renamed KeeperStaking to KeeperDogfood and added methods related to dogfood types. |
x/oracle/keeper/keeper.go |
Refactored references from KeeperStaking to KeeperDogfood within the Keeper struct and the NewKeeper function. |
x/oracle/keeper/msg_server_create_price_test.go |
Updated imports, method references, and field types from staking to dogfood module in the price creation tests. |
x/oracle/keeper/single.go |
Refactored validator power retrieval from staking module to dogfood module in recacheAggregatorContext and initAggregatorContext . |
x/oracle/types/message_create_price.go |
Switched from sdk.ValAddressFromBech32 to sdk.AccAddressFromBech32 for creator validation in MsgCreatePrice . |
x/oracle/types/message_create_price_test.go |
Updated Creator field type in TestMsgCreatePrice_ValidateBasic from ValAddress to AccAddress . |
sequenceDiagram
participant Client
participant CosmosSDK
participant DogfoodModule
participant GasMeter
Client->>CosmosSDK: Submit Transaction
CosmosSDK->>DogfoodModule: Validate Signatures
DogfoodModule-->>CosmosSDK: Validation Result
CosmosSDK->>GasMeter: Consume Gas
GasMeter-->>CosmosSDK: Gas Consumption
CosmosSDK->>Client: Transaction Result
In the Cosmos, valves rotate,
Validator keys, they resonate.
Dogfood modules take the flame,
Gas is burned in code’s grand game.
With signatures now verified,
Transactions flow, no need to hide.
In this upgrade, success resides! 🚀🐇🌟
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?
@leonz789 could you fix the lint error?
Test passed.
Description
Use consensuskey as signing key to sign create-price message, and add corresponding support for ed25519. This PR need to work with PR #63 . It should be merged after PR #63
Changes
Notable changes:
Closes #XXX
Summary by CodeRabbit
New Features
Improvements
Compatibility Updates