Skip to content

Commit c2bf27c

Browse files
mergify[bot]catShaark
andauthoredOct 27, 2022
use controller module address instead of module name for NewMsgChannelOpenInit (#2568) (#2614)
## Description In controller keeper's `registerInterchainAccount` we execute a `MsgChannelOpenInit` to set up an ica channel. Currently, we have [`icatypes.ModuleName`](https://github.com/cosmos/ibc-go/blob/95cec44c9fb10a2e84a327cdd98c7d588f091f42/modules/apps/27-interchain-accounts/controller/keeper/account.go#L63) as the msg signer. This is not good because the [`ValidateBasic()`](https://github.com/cosmos/ibc-go/blob/692790402a033c1d9647c1e1eded5351332283bf/modules/core/04-channel/types/msgs.go#L34) of `MsgChannelOpenInit` has check if msg signer is bech32 or not. ref #2566 closes: #2559 --- Before we can merge this PR, please make sure that all the following items have been checked off. If any of the checklist items are not applicable, please leave them but write a little note why. - [ ] Targeted PR against correct branch (see [CONTRIBUTING.md](https://github.com/cosmos/ibc-go/blob/master/CONTRIBUTING.md#pr-targeting)) - [ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work. - [ ] Code follows the [module structure standards](https://github.com/cosmos/cosmos-sdk/blob/main/docs/docs/building-modules/10-structure.md). - [ ] Wrote unit and integration [tests](https://github.com/cosmos/ibc-go/blob/master/CONTRIBUTING.md#testing) - [ ] Updated relevant documentation (`docs/`) or specification (`x/<module>/spec/`) - [ ] Added relevant `godoc` [comments](https://blog.golang.org/godoc-documenting-go-code). - [ ] Added a relevant changelog entry to the `Unreleased` section in `CHANGELOG.md` - [ ] Re-reviewed `Files changed` in the Github PR explorer - [ ] Review `Codecov Report` in the comment section below once CI passes (cherry picked from commit 6105db4) Co-authored-by: khanh-notional <50263489+catShaark@users.noreply.github.com>
1 parent 8022d0b commit c2bf27c

File tree

1 file changed

+2
-1
lines changed
  • modules/apps/27-interchain-accounts/controller/keeper

1 file changed

+2
-1
lines changed
 

‎modules/apps/27-interchain-accounts/controller/keeper/account.go

+2-1
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@ package keeper
33
import (
44
sdk "github.com/cosmos/cosmos-sdk/types"
55
sdkerrors "github.com/cosmos/cosmos-sdk/types/errors"
6+
authtypes "github.com/cosmos/cosmos-sdk/x/auth/types"
67

78
icatypes "github.com/cosmos/ibc-go/v6/modules/apps/27-interchain-accounts/types"
89
channeltypes "github.com/cosmos/ibc-go/v6/modules/core/04-channel/types"
@@ -60,7 +61,7 @@ func (k Keeper) registerInterchainAccount(ctx sdk.Context, connectionID, portID,
6061
}
6162
}
6263

63-
msg := channeltypes.NewMsgChannelOpenInit(portID, version, channeltypes.ORDERED, []string{connectionID}, icatypes.HostPortID, icatypes.ModuleName)
64+
msg := channeltypes.NewMsgChannelOpenInit(portID, version, channeltypes.ORDERED, []string{connectionID}, icatypes.HostPortID, authtypes.NewModuleAddress(icatypes.ModuleName).String())
6465
handler := k.msgRouter.Handler(msg)
6566
res, err := handler(ctx, msg)
6667
if err != nil {

0 commit comments

Comments
 (0)