Skip to content

Commit 57cea83

Browse files
0xLaurenzomergify[bot]
authored andcommitted
remove port prefix requirement (#2590)
* remove port prefix requirement * chore: remove depcrated test and fix lint * add changelog entry * Update CHANGELOG.md Co-authored-by: Damian Nolan <damiannolan@gmail.com> Co-authored-by: Damian Nolan <damiannolan@gmail.com> (cherry picked from commit 5f9966b) # Conflicts: # modules/apps/27-interchain-accounts/host/keeper/handshake.go
1 parent b15b372 commit 57cea83

File tree

3 files changed

+4
-8
lines changed

3 files changed

+4
-8
lines changed

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,7 @@ Ref: https://keepachangelog.com/en/1.0.0/
4242

4343
### State Machine Breaking
4444

45+
* (27-interchain-accounts) [\#2580](https://github.com/cosmos/ibc-go/issues/2580) Removing port prefix requirement from the ICA host channel handshake
4546
* (transfer) [\#2377](https://github.com/cosmos/ibc-go/pull/2377) Adding `sequence` to `MsgTransferResponse`.
4647

4748
### Improvements

modules/apps/27-interchain-accounts/host/keeper/handshake.go

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@ package keeper
22

33
import (
44
"fmt"
5-
"strings"
65

76
sdk "github.com/cosmos/cosmos-sdk/types"
87
sdkerrors "github.com/cosmos/cosmos-sdk/types/errors"
@@ -35,10 +34,13 @@ func (k Keeper) OnChanOpenTry(
3534
return "", sdkerrors.Wrapf(icatypes.ErrInvalidHostPort, "expected %s, got %s", icatypes.PortID, portID)
3635
}
3736

37+
<<<<<<< HEAD
3838
if !strings.HasPrefix(counterparty.PortId, icatypes.PortPrefix) {
3939
return "", sdkerrors.Wrapf(icatypes.ErrInvalidControllerPort, "expected %s{owner-account-address}, got %s", icatypes.PortPrefix, counterparty.PortId)
4040
}
4141

42+
=======
43+
>>>>>>> 5f9966b (remove port prefix requirement (#2590))
4244
var metadata icatypes.Metadata
4345
if err := icatypes.ModuleCdc.UnmarshalJSON([]byte(counterpartyVersion), &metadata); err != nil {
4446
return "", sdkerrors.Wrapf(icatypes.ErrUnknownDataType, "cannot unmarshal ICS-27 interchain accounts metadata")

modules/apps/27-interchain-accounts/host/keeper/handshake_test.go

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -171,13 +171,6 @@ func (suite *KeeperTestSuite) TestOnChanOpenTry() {
171171
},
172172
false,
173173
},
174-
{
175-
"invalid counterparty port ID",
176-
func() {
177-
channel.Counterparty.PortId = "invalid-port-id"
178-
},
179-
false,
180-
},
181174
{
182175
"connection not found",
183176
func() {

0 commit comments

Comments
 (0)