add check send enabled (cosmos#2679)

* add check send enabled * changelog
dymensionxyz · Nov 4, 2022 · b1f494c · b1f494c
1 parent f69761d
commit b1f494c
Show file tree

Hide file tree

Showing 4 changed files with 29 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -122,6 +122,7 @@ Ref: https://keepachangelog.com/en/1.0.0/
 * (light-clients/07-tendermint) [\#1674](https://github.com/cosmos/ibc-go/pull/1674) Submitted ClientState is zeroed out before checking the proof in order to prevent the proposal from containing information governance is not actually voting on.
 * (modules/core/02-client)[\#1676](https://github.com/cosmos/ibc-go/pull/1676) ClientState must be zeroed out for `UpgradeProposals` to pass validation. This prevents a proposal containing information governance is not actually voting on.
 * (modules/core/keeper) [\#2403](https://github.com/cosmos/ibc-go/pull/2403) Added a function in keeper to cater for blank pointers.
+* (apps/transfer) [\#2672](https://github.com/cosmos/ibc-go/pull/2672) Check `x/bank` send enabled.
 
 ## [v5.0.1](https://github.com/cosmos/ibc-go/releases/tag/v5.0.1) - 2022-10-27
 

diff --git a/modules/apps/transfer/keeper/msg_server.go b/modules/apps/transfer/keeper/msg_server.go
@@ -24,6 +24,10 @@ func (k Keeper) Transfer(goCtx context.Context, msg *types.MsgTransfer) (*types.
  return nil, err
  }
 
+ if !k.bankKeeper.IsSendEnabledCoin(ctx, msg.Token) {
+ return nil, sdkerrors.Wrapf(types.ErrSendDisabled, "%s transfers are currently disabled", msg.Token.Denom)
+ }
+
  if k.bankKeeper.BlockedAddr(sender) {
  return nil, sdkerrors.Wrapf(sdkerrors.ErrUnauthorized, "%s is not allowed to send funds", sender)
  }

diff --git a/modules/apps/transfer/keeper/msg_server_test.go b/modules/apps/transfer/keeper/msg_server_test.go
@@ -2,6 +2,7 @@ package keeper_test
 
 import (
  sdk "github.com/cosmos/cosmos-sdk/types"
+ banktypes "github.com/cosmos/cosmos-sdk/x/bank/types"
 
  "github.com/cosmos/ibc-go/v6/modules/apps/transfer/types"
 )
@@ -19,6 +20,17 @@ func (suite *KeeperTestSuite) TestMsgTransfer() {
  func() {},
  true,
  },
+ {
+ "bank send enabled for denom",
+ func() {
+ suite.chainA.GetSimApp().BankKeeper.SetParams(suite.chainA.GetContext(),
+ banktypes.Params{
+ SendEnabled: []*banktypes.SendEnabled{{Denom: sdk.DefaultBondDenom, Enabled: true}},
+ },
+ )
+ },
+ true,
+ },
  {
  "send transfers disabled",
  func() {
@@ -44,6 +56,17 @@ func (suite *KeeperTestSuite) TestMsgTransfer() {
  },
  false,
  },
+ {
+ "bank send disabled for denom",
+ func() {
+ suite.chainA.GetSimApp().BankKeeper.SetParams(suite.chainA.GetContext(),
+ banktypes.Params{
+ SendEnabled: []*banktypes.SendEnabled{{Denom: sdk.DefaultBondDenom, Enabled: false}},
+ },
+ )
+ },
+ false,
+ },
  {
  "channel does not exist",
  func() {

diff --git a/modules/apps/transfer/types/expected_keepers.go b/modules/apps/transfer/types/expected_keepers.go
@@ -24,6 +24,7 @@ type BankKeeper interface {
  SendCoinsFromModuleToAccount(ctx sdk.Context, senderModule string, recipientAddr sdk.AccAddress, amt sdk.Coins) error
  SendCoinsFromAccountToModule(ctx sdk.Context, senderAddr sdk.AccAddress, recipientModule string, amt sdk.Coins) error
  BlockedAddr(addr sdk.AccAddress) bool
+ IsSendEnabledCoin(ctx sdk.Context, coin sdk.Coin) bool
 }
 
 // ChannelKeeper defines the expected IBC channel keeper