--- title: IBC-Go v2 to v3 sidebar_label: IBC-Go v2 to v3 sidebar_position: 4 slug: /migrations/v2-to-v3 --- # Migrating from ibc-go v2 to v3 This document is intended to highlight significant changes which may require more information than presented in the CHANGELOG. Any changes that must be done by a user of ibc-go should be documented here. There are four sections based on the four potential user groups of this document: - Chains - IBC Apps - Relayers - IBC Light Clients **Note:** ibc-go supports golang semantic versioning and therefore all imports must be updated to bump the version number on major releases. ```go github.com/cosmos/ibc-go/v2 -> github.com/cosmos/ibc-go/v3 ``` No genesis or in-place migrations are required when upgrading from v1 or v2 of ibc-go. ## Chains ### ICS20 The `transferkeeper.NewKeeper(...)` now takes in an ICS4Wrapper. The ICS4Wrapper should be the IBC Channel Keeper unless ICS 20 is being connected to a middleware application. ### ICS27 ICS27 Interchain Accounts has been added as a supported IBC application of ibc-go. Please see the [ICS27 documentation](../02-apps/02-interchain-accounts/01-overview.md) for more information. ### Upgrade Proposal If the chain will adopt ICS27, it must set the appropriate params during the execution of the upgrade handler in `app.go`: ```go app.UpgradeKeeper.SetUpgradeHandler("v3", func(ctx sdk.Context, _ upgradetypes.Plan, fromVM module.VersionMap) (module.VersionMap, error) { // set the ICS27 consensus version so InitGenesis is not run fromVM[icatypes.ModuleName] = icamodule.ConsensusVersion() // create ICS27 Controller submodule params controllerParams := icacontrollertypes.Params{ ControllerEnabled: true, } // create ICS27 Host submodule params hostParams := icahosttypes.Params{ HostEnabled: true, AllowMessages: []string{"/cosmos.bank.v1beta1.MsgSend", ...}, } // initialize ICS27 module icamodule.InitModule(ctx, controllerParams, hostParams) ... return app.mm.RunMigrations(ctx, app.configurator, fromVM) }) ``` The host and controller submodule params only need to be set if the chain integrates those submodules. For example, if a chain chooses not to integrate a controller submodule, it may pass empty params into `InitModule`. #### Add `StoreUpgrades` for ICS27 module For ICS27 it is also necessary to [manually add store upgrades](https://docs.cosmos.network/main/learn/advanced/upgrade#add-storeupgrades-for-new-modules) for the new ICS27 module and then configure the store loader to apply those upgrades in `app.go`: ```go if upgradeInfo.Name == "v3" && !app.UpgradeKeeper.IsSkipHeight(upgradeInfo.Height) { storeUpgrades := store.StoreUpgrades{ Added: []string{icacontrollertypes.StoreKey, icahosttypes.StoreKey}, } app.SetStoreLoader(upgradetypes.UpgradeStoreLoader(upgradeInfo.Height, &storeUpgrades)) } ``` This ensures that the new module's stores are added to the multistore before the migrations begin. The host and controller submodule keys only need to be added if the chain integrates those submodules. For example, if a chain chooses not to integrate a controller submodule, it does not need to add the controller key to the `Added` field. ### Genesis migrations If the chain will adopt ICS27 and chooses to upgrade via a genesis export, then the ICS27 parameters must be set during genesis migration. The migration code required may look like: ```go controllerGenesisState := icatypes.DefaultControllerGenesis() // overwrite parameters as desired controllerGenesisState.Params = icacontrollertypes.Params{ ControllerEnabled: true, } hostGenesisState := icatypes.DefaultHostGenesis() // overwrite parameters as desired hostGenesisState.Params = icahosttypes.Params{ HostEnabled: true, AllowMessages: []string{"/cosmos.bank.v1beta1.MsgSend", ...}, } icaGenesisState := icatypes.NewGenesisState(controllerGenesisState, hostGenesisState) // set new ics27 genesis state appState[icatypes.ModuleName] = clientCtx.JSONCodec.MustMarshalJSON(icaGenesisState) ``` ### Ante decorator The field of type `channelkeeper.Keeper` in the `AnteDecorator` structure has been replaced with a field of type `*keeper.Keeper`: ```diff type AnteDecorator struct { - k channelkeeper.Keeper + k *keeper.Keeper } - func NewAnteDecorator(k channelkeeper.Keeper) AnteDecorator { + func NewAnteDecorator(k *keeper.Keeper) AnteDecorator { return AnteDecorator{k: k} } ``` ## IBC Apps ### `OnChanOpenTry` must return negotiated application version The `OnChanOpenTry` application callback has been modified. The return signature now includes the application version. IBC applications must perform application version negotiation in `OnChanOpenTry` using the counterparty version. The negotiated application version then must be returned in `OnChanOpenTry` to core IBC. Core IBC will set this version in the TRYOPEN channel. ### `OnChanOpenAck` will take additional `counterpartyChannelID` argument The `OnChanOpenAck` application callback has been modified. The arguments now include the counterparty channel id. ### `NegotiateAppVersion` removed from `IBCModule` interface Previously this logic was handled by the `NegotiateAppVersion` function. Relayers would query this function before calling `ChanOpenTry`. Applications would then need to verify that the passed in version was correct. Now applications will perform this version negotiation during the channel handshake, thus removing the need for `NegotiateAppVersion`. ### Channel state will not be set before application callback The channel handshake logic has been reorganized within core IBC. Channel state will not be set in state after the application callback is performed. Applications must rely only on the passed in channel parameters instead of querying the channel keeper for channel state. ### IBC application callbacks moved from `AppModule` to `IBCModule` Previously, IBC module callbacks were apart of the `AppModule` type. The recommended approach is to create an `IBCModule` type and move the IBC module callbacks from `AppModule` to `IBCModule` in a separate file `ibc_module.go`. The mock module go API has been broken in this release by applying the above format. The IBC module callbacks have been moved from the mock modules `AppModule` into a new type `IBCModule`. As apart of this release, the mock module now supports middleware testing. Please see the [README](https://github.com/cosmos/ibc-go/blob/v4.4.0/testing/README.md#middleware-testing) for more information. Please review the [mock](https://github.com/cosmos/ibc-go/blob/v4.4.0/testing/mock/ibc_module.go) and [transfer](https://github.com/cosmos/ibc-go/blob/v4.4.0/modules/apps/transfer/ibc_module.go) modules as examples. Additionally, [simapp](https://github.com/cosmos/ibc-go/blob/v4.4.0/testing/simapp/app.go) provides an example of how `IBCModule` types should now be added to the IBC router in favour of `AppModule`. ### IBC testing package `TestChain`s are now created with chainID's beginning from an index of 1. Any calls to `GetChainID(0)` will now fail. Please increment all calls to `GetChainID` by 1. ## Relayers `AppVersion` gRPC has been removed. The `version` string in `MsgChanOpenTry` has been deprecated and will be ignored by core IBC. Relayers no longer need to determine the version to use on the `ChanOpenTry` step. IBC applications will determine the correct version using the counterparty version. ## IBC Light Clients The `GetProofSpecs` function has been removed from the `ClientState` interface. This function was previously unused by core IBC. Light clients which don't use this function may remove it.