Skip to content

Commit 2138221

Browse files
mmsqejtieri
andauthored
fix: reduce get bech32 prefix when get signer (#1231)
* add missing getFeePayer for clienttypes * add missing getFeePayer for feetypes * apply in penumbra * add change doc --------- Co-authored-by: Justin Tieri <[email protected]>
1 parent d878a55 commit 2138221

File tree

3 files changed

+22
-0
lines changed

3 files changed

+22
-0
lines changed

CHANGELOG.md

+1
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@
2121
* [\#1208](https://github.com/cosmos/relayer/pull/1208) Replace gogo/protobuf to cosmos/gogoproto.
2222
* [\#1221](https://github.com/cosmos/relayer/pull/1221) Update cometbft to v0.37.2 and ibc-go to v7.2.0.
2323
* [\#1226](https://github.com/cosmos/relayer/pull/1226) Avoid invalid Bech32 prefix error in parallel tests when sdk Config get overwritten by each other in single process.
24+
* [\#1231](https://github.com/cosmos/relayer/pull/1231) Reduce get bech32 prefix when get signer.
2425

2526
## v0.9.3
2627

relayer/chains/cosmos/log.go

+8
Original file line numberDiff line numberDiff line change
@@ -164,11 +164,19 @@ func getFeePayer(tx *typestx.Tx) string {
164164
case *clienttypes.MsgUpdateClient:
165165
// Same failure mode as MsgCreateClient.
166166
return firstMsg.Signer
167+
case *clienttypes.MsgUpgradeClient:
168+
return firstMsg.Signer
167169
case *clienttypes.MsgSubmitMisbehaviour:
168170
// Same failure mode as MsgCreateClient.
169171
return firstMsg.Signer
172+
case *feetypes.MsgRegisterPayee:
173+
return firstMsg.Relayer
170174
case *feetypes.MsgRegisterCounterpartyPayee:
171175
return firstMsg.Relayer
176+
case *feetypes.MsgPayPacketFee:
177+
return firstMsg.Signer
178+
case *feetypes.MsgPayPacketFeeAsync:
179+
return firstMsg.PacketFee.RefundAddress
172180
default:
173181
return firstMsg.GetSigners()[0].String()
174182
}

relayer/chains/penumbra/log.go

+13
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ import (
66
"github.com/cosmos/cosmos-sdk/codec/types"
77
sdk "github.com/cosmos/cosmos-sdk/types"
88
typestx "github.com/cosmos/cosmos-sdk/types/tx"
9+
feetypes "github.com/cosmos/ibc-go/v7/modules/apps/29-fee/types"
910
transfertypes "github.com/cosmos/ibc-go/v7/modules/apps/transfer/types"
1011
clienttypes "github.com/cosmos/ibc-go/v7/modules/core/02-client/types"
1112
"github.com/cosmos/relayer/v2/relayer/provider"
@@ -151,6 +152,18 @@ func getFeePayer(tx *typestx.Tx) string {
151152
case *clienttypes.MsgUpdateClient:
152153
// Same failure mode as MsgCreateClient.
153154
return firstMsg.Signer
155+
case *clienttypes.MsgUpgradeClient:
156+
return firstMsg.Signer
157+
case *clienttypes.MsgSubmitMisbehaviour:
158+
return firstMsg.Signer
159+
case *feetypes.MsgRegisterPayee:
160+
return firstMsg.Relayer
161+
case *feetypes.MsgRegisterCounterpartyPayee:
162+
return firstMsg.Relayer
163+
case *feetypes.MsgPayPacketFee:
164+
return firstMsg.Signer
165+
case *feetypes.MsgPayPacketFeeAsync:
166+
return firstMsg.PacketFee.RefundAddress
154167
default:
155168
return firstMsg.GetSigners()[0].String()
156169
}

0 commit comments

Comments
 (0)