From 232113f6c1d7576cdf5b20b40b3277b3e30c0990 Mon Sep 17 00:00:00 2001 From: beer-1 <147697694+beer-1@users.noreply.github.com> Date: Wed, 6 Mar 2024 18:59:26 +0900 Subject: [PATCH] fix readme --- app/ibc-middleware/README.md | 163 ++++++++++++++++------------------- 1 file changed, 74 insertions(+), 89 deletions(-) diff --git a/app/ibc-middleware/README.md b/app/ibc-middleware/README.md index 837485b..41784ce 100644 --- a/app/ibc-middleware/README.md +++ b/app/ibc-middleware/README.md @@ -1,137 +1,122 @@ # IBC-hooks -This module is copied from [osmosis](https://github.com/osmosis-labs/osmosis) and changed to execute move contract with ICS-20 token transfer calls. +## Wasm Hooks -## Move Hooks - -The move hook is an IBC middleware which is used to allow ICS-20 token transfers to initiate contract calls. +The wasm hook is an IBC middleware which is used to allow ICS-20 token transfers to initiate contract calls. This allows cross-chain contract calls, that involve token movement. This is useful for a variety of usecases. One of primary importance is cross-chain swaps, which is an extremely powerful primitive. The mechanism enabling this is a `memo` field on every ICS20 transfer packet as of [IBC v3.4.0](https://medium.com/the-interchain-foundation/moving-beyond-simple-token-transfers-d42b2b1dc29b). -Move hooks is an IBC middleware that parses an ICS20 transfer, and if the `memo` field is of a particular form, executes a move contract call. We now detail the `memo` format for `move` contract calls, and the execution guarantees provided. +Wasm hooks is an IBC middleware that parses an ICS20 transfer, and if the `memo` field is of a particular form, executes a wasm contract call. We now detail the `memo` format for `wasm` contract calls, and the execution guarantees provided. -### Move Contract Execution Format +### Cosmwasm Contract Execution Format -Before we dive into the IBC metadata format, we show the move execute message format, so the reader has a sense of what are the fields we need to be setting in. -The move `MsgExecute` is defined [here](https://github.com/initia-labs/initia/blob/27ceb2b535bfe355062b6770ebd1463aa0028ffc/x/move/types/tx.pb.go#L119-L135) as the following type: +Before we dive into the IBC metadata format, we show the cosmwasm execute message format, so the reader has a sense of what are the fields we need to be setting in. +The cosmwasm `MsgExecuteContract` is defined [here](https://github.com/CosmWasm/wasmd/blob/4fe2fbc8f322efdaf187e2e5c99ce32fd1df06f0/x/wasm/types/tx.pb.go#L340-L349 +) as the following type: ```go -type MsgExecute struct { - // Sender is the that actor that signed the messages - Sender string `protobuf:"bytes,1,opt,name=sender,proto3" json:"sender,omitempty"` - // ModuleAddress is the address of the module deployer - ModuleAddress string `protobuf:"bytes,2,opt,name=module_address,json=moduleAddress,proto3" json:"module_address,omitempty"` - // ModuleName is the name of module to execute - ModuleName string `protobuf:"bytes,3,opt,name=module_name,json=moduleName,proto3" json:"module_name,omitempty"` - // FunctionName is the name of a function to execute - FunctionName string `protobuf:"bytes,4,opt,name=function_name,json=functionName,proto3" json:"function_name,omitempty"` - // TypeArgs is the type arguments of a function to execute - // ex) "0x1::BasicCoin::Initia", "bool", "u8", "u64" - TypeArgs []string `protobuf:"bytes,5,rep,name=type_args,json=typeArgs,proto3" json:"type_args,omitempty"` - // Args is the arguments of a function to execute - // - number: little endian - // - string: base64 bytes - Args [][]byte `protobuf:"bytes,6,rep,name=args,proto3" json:"args,omitempty"` +type MsgExecuteContract struct { + // Sender is the actor that committed the message in the sender chain + Sender string + // Contract is the address of the smart contract + Contract string + // Msg json encoded message to be passed to the contract + Msg RawContractMessage + // Funds coins that are transferred to the contract on execution + Funds sdk.Coins } ``` So we detail where we want to get each of these fields from: -- Sender: We cannot trust the sender of an IBC packet, the counter-party chain has full ability to lie about it. - We cannot risk this sender being confused for a particular user or module address on Initia. - So we replace the sender with an account to represent the sender prefixed by the channel and a move module prefix. - This is done by setting the sender to `Bech32(Hash(Hash("ibc-hook-intermediary") + channelID/sender))`, where the channelId is the channel id on the local chain. -- ModuleAddress: This field should be directly obtained from the ICS-20 packet metadata -- ModuleName: This field should be directly obtained from the ICS-20 packet metadata -- FunctionName: This field should be directly obtained from the ICS-20 packet metadata -- TypeArgs: This field should be directly obtained from the ICS-20 packet metadata -- Args: This field should be directly obtained from the ICS-20 packet metadata. +* Sender: We cannot trust the sender of an IBC packet, the counterparty chain has full ability to lie about it. +We cannot risk this sender being confused for a particular user or module address on Osmosis. +So we replace the sender with an account to represent the sender prefixed by the channel and a wasm module prefix. +This is done by setting the sender to `Bech32(Hash("ibc-wasm-hook-intermediary" || channelID || sender))`, where the channelId is the channel id on the local chain. +* Contract: This field should be directly obtained from the ICS-20 packet metadata +* Msg: This field should be directly obtained from the ICS-20 packet metadata. +* Funds: This field is set to the amount of funds being sent over in the ICS 20 packet. One detail is that the denom in the packet is the counterparty chains representation of the denom, so we have to translate it to Osmosis' representation. + +> **_WARNING:_** Due to a [bug](https://twitter.com/SCVSecurity/status/1682329758020022272) in the packet forward middleware, we cannot trust the sender from chains that use PFM. Until that is fixed, we recommend chains to not trust the sender on contracts executed via IBC hooks. -So our constructed move message that we execute will look like: +So our constructed cosmwasm message that we execute will look like: ```go msg := MsgExecuteContract{ - // Sender is the that actor that signed the messages - Sender: "init1-hash-of-channel-and-sender", - // ModuleAddress is the address of the module deployer - ModuleAddress: packet.data.memo["move"]["module_address"], - // ModuleName is the name of module to execute - ModuleName: packet.data.memo["move"]["module_name"], - // FunctionName is the name of a function to execute - FunctionName: packet.data.memo["move"]["function_name"], - // TypeArgs is the type arguments of a function to execute - // ex) "0x1::BasicCoin::Initia", "bool", "u8", "u64" - TypeArgs: packet.data.memo["move"]["type_args"], - // Args is the arguments of a function to execute - // - number: little endian - // - string: base64 bytes - Args: packet.data.memo["move"]["args"]} + // Sender is the that actor that signed the messages + Sender: "osmo1-hash-of-channel-and-sender", + // Contract is the address of the smart contract + Contract: packet.data.memo["wasm"]["ContractAddress"], + // Msg json encoded message to be passed to the contract + Msg: packet.data.memo["wasm"]["Msg"], + // Funds coins that are transferred to the contract on execution + Funds: sdk.NewCoin{Denom: ibc.ConvertSenderDenomToLocalDenom(packet.data.Denom), Amount: packet.data.Amount} ``` ### ICS20 packet structure -So given the details above, we propogate the implied ICS20 packet data structure. +So given the details above, we propagate the implied ICS20 packet data structure. ICS20 is JSON native, so we use JSON for the memo format. ```json { - //... other ibc fields that we don't care about - "data": { - "denom": "denom on counterparty chain (e.g. uatom)", // will be transformed to the local denom (ibc/...) - "amount": "1000", - "sender": "addr on counterparty chain", // will be transformed - "receiver": "ModuleAddr::ModuleName::FunctionName", - "memo": { - "move": { - "module_address": "0x1", - "module_name": "dex", - "function_name": "swap", - "type_args": ["0x1::native_uinit::Coin", "0x1::native_uusdc::Coin"], - "args": ["base64 encoded bytes array"] - } + //... other ibc fields that we don't care about + "data":{ + "denom": "denom on counterparty chain (e.g. uatom)", // will be transformed to the local denom (ibc/...) + "amount": "1000", + "sender": "addr on counterparty chain", // will be transformed + "receiver": "contract addr or blank", + "memo": { + "wasm": { + "contract": "osmo1contractAddr", + "msg": { + "raw_message_fields": "raw_message_data", + } + } + } } - } } ``` -An ICS20 packet is formatted correctly for movehooks iff the following all hold: +An ICS20 packet is formatted correctly for wasmhooks iff the following all hold: -- `memo` is not blank -- `memo` is valid JSON -- `memo` has at least one key, with value `"move"` -- `memo["move"]` has exactly five entries, `"module_address"`, `"module_name"`, `"function_name"`, `"type_args"` and `"args"` -- `receiver` == "" || `receiver` == "module_address::module_name::function_name" +* `memo` is not blank +* `memo` is valid JSON +* `memo` has at least one key, with value `"wasm"` +* `memo["wasm"]` has exactly two entries, `"contract"` and `"msg"` +* `memo["wasm"]["msg"]` is a valid JSON object +* `receiver == "" || receiver == memo["wasm"]["contract"]` -We consider an ICS20 packet as directed towards movehooks iff all of the following hold: +We consider an ICS20 packet as directed towards wasmhooks iff all of the following hold: -- `memo` is not blank -- `memo` is valid JSON -- `memo` has at least one key, with name `"move"` +* `memo` is not blank +* `memo` is valid JSON +* `memo` has at least one key, with name `"wasm"` -If an ICS20 packet is not directed towards movehooks, movehooks doesn't do anything. -If an ICS20 packet is directed towards movehooks, and is formatted incorrectly, then movehooks returns an error. +If an ICS20 packet is not directed towards wasmhooks, wasmhooks doesn't do anything. +If an ICS20 packet is directed towards wasmhooks, and is formatted incorrectly, then wasmhooks returns an error. ### Execution flow -Pre move hooks: +Pre wasm hooks: -- Ensure the incoming IBC packet is cryptogaphically valid -- Ensure the incoming IBC packet is not timed out. +* Ensure the incoming IBC packet is cryptogaphically valid +* Ensure the incoming IBC packet is not timed out. -In move hooks, pre packet execution: +In Wasm hooks, pre packet execution: -- Ensure the packet is correctly formatted (as defined above) -- Edit the receiver to be the hardcoded IBC module account +* Ensure the packet is correctly formatted (as defined above) +* Edit the receiver to be the hardcoded IBC module account -In move hooks, post packet execution: +In wasm hooks, post packet execution: -- Construct move message as defined before -- Execute move message -- if move message has error, return ErrAck -- otherwise continue through middleware +* Construct wasm message as defined before +* Execute wasm message +* if wasm message has error, return ErrAck +* otherwise continue through middleware -# Testing strategy +## Testing strategy See go tests.