feat: add gov spam prevention antehandler

ante/ante.go

@@ -1,9 +1,11 @@
 package ante
 
 import (
+	"github.com/cosmos/cosmos-sdk/codec"
 	sdk "github.com/cosmos/cosmos-sdk/types"
 	sdkerrors "github.com/cosmos/cosmos-sdk/types/errors"
 	"github.com/cosmos/cosmos-sdk/x/auth/ante"
+	govkeeper "github.com/cosmos/cosmos-sdk/x/gov/keeper"
 	paramtypes "github.com/cosmos/cosmos-sdk/x/params/types"
 	ibcante "github.com/cosmos/ibc-go/v4/modules/core/ante"
 	ibckeeper "github.com/cosmos/ibc-go/v4/modules/core/keeper"
@@ -15,6 +17,8 @@ import (
 // channel keeper.
 type HandlerOptions struct {
 	ante.HandlerOptions
+	Codec                codec.BinaryCodec
+	GovKeeper            *govkeeper.Keeper
 	IBCkeeper            *ibckeeper.Keeper
 	BypassMinFeeMsgTypes []string
 	GlobalFeeSubspace    paramtypes.Subspace
@@ -34,6 +38,9 @@ func NewAnteHandler(opts HandlerOptions) (sdk.AnteHandler, error) {
 	if opts.IBCkeeper == nil {
 		return nil, sdkerrors.Wrap(sdkerrors.ErrLogic, "IBC keeper is required for AnteHandler")
 	}
+	if opts.GovKeeper == nil {
+		return nil, sdkerrors.Wrap(sdkerrors.ErrLogic, "gov keeper is required for AnteHandler")
+	}
 	if opts.GlobalFeeSubspace.Name() == "" {
 		return nil, sdkerrors.Wrap(sdkerrors.ErrNotFound, "globalfee param store is required for AnteHandler")
 	}
@@ -60,7 +67,7 @@ func NewAnteHandler(opts HandlerOptions) (sdk.AnteHandler, error) {
 		ante.NewValidateMemoDecorator(opts.AccountKeeper),
 		ante.NewConsumeGasForTxSizeDecorator(opts.AccountKeeper),
 		gaiafeeante.NewFeeDecorator(opts.BypassMinFeeMsgTypes, opts.GlobalFeeSubspace, opts.StakingSubspace, maxBypassMinFeeMsgGasUsage),
-
+		NewGovPreventSpamDecorator(opts.Codec, opts.GovKeeper),
 		ante.NewDeductFeeDecorator(opts.AccountKeeper, opts.BankKeeper, opts.FeegrantKeeper),
 		ante.NewSetPubKeyDecorator(opts.AccountKeeper), // SetPubKeyDecorator must be called before all signature verification decorators
 		ante.NewValidateSigCountDecorator(opts.AccountKeeper),

ante/gov_ante.go

@@ -0,0 +1,90 @@
+package ante
+
+import (
+	"github.com/cosmos/cosmos-sdk/codec"
+	sdk "github.com/cosmos/cosmos-sdk/types"
+	sdkerrors "github.com/cosmos/cosmos-sdk/types/errors"
+	"github.com/cosmos/cosmos-sdk/x/authz"
+	govkeeper "github.com/cosmos/cosmos-sdk/x/gov/keeper"
+	govtypes "github.com/cosmos/cosmos-sdk/x/gov/types"
+)
+
+var MiniumInitialDepositRate = sdk.NewDecWithPrec(10, 2)
+
+type GovPreventSpamDecorator struct {
+	govKeeper *govkeeper.Keeper
+	cdc       codec.BinaryCodec
+}
+
+func NewGovPreventSpamDecorator(cdc codec.BinaryCodec, govKeeper *govkeeper.Keeper) GovPreventSpamDecorator {
+	return GovPreventSpamDecorator{
+		govKeeper: govKeeper,
+		cdc:       cdc,
+	}
+}
+
+func (gpsd GovPreventSpamDecorator) AnteHandle(
+	ctx sdk.Context, tx sdk.Tx,
+	simulate bool, next sdk.AnteHandler,
+) (newCtx sdk.Context, err error) {
+	msgs := tx.GetMsgs()
+	if err = gpsd.checkSpamSubmitProposalMsg(ctx, msgs); err != nil {
+		return ctx, err
+	}
+
+	return next(ctx, tx, simulate)
+}
+
+func (gpsd GovPreventSpamDecorator) checkSpamSubmitProposalMsg(ctx sdk.Context, msgs []sdk.Msg) error {
+	validMsg := func(m sdk.Msg) error {
+		if msg, ok := m.(*govtypes.MsgSubmitProposal); ok {
+			// prevent spam gov msg
+			depositParams := gpsd.govKeeper.GetDepositParams(ctx)
+			miniumInitialDeposit := gpsd.calcMiniumInitialDeposit(depositParams.MinDeposit)
+			if msg.InitialDeposit.IsAllLT(miniumInitialDeposit) {
+				return sdkerrors.Wrapf(sdkerrors.ErrInsufficientFunds, "not enough initial deposit. required: %v", miniumInitialDeposit)
+			}
+		}
+
+		return nil
+	}
+
+	validAuthz := func(execMsg *authz.MsgExec) error {
+		for _, v := range execMsg.Msgs {
+			var innerMsg sdk.Msg
+			if err := gpsd.cdc.UnpackAny(v, &innerMsg); err != nil {
+				return sdkerrors.Wrapf(sdkerrors.ErrUnauthorized, "cannot unmarshal authz exec msgs")
+			}
+
+			if err := validMsg(innerMsg); err != nil {
+				return err
+			}
+		}
+
+		return nil
+	}
+
+	for _, m := range msgs {
+		if msg, ok := m.(*authz.MsgExec); ok {
+			if err := validAuthz(msg); err != nil {
+				return err
+			}
+			continue
+		}
+
+		// validate normal msgs
+		if err := validMsg(m); err != nil {
+			return err
+		}
+	}
+	return nil
+}
+
+func (gpsd GovPreventSpamDecorator) calcMiniumInitialDeposit(minDeposit sdk.Coins) (miniumInitialDeposit sdk.Coins) {
+	for _, coin := range minDeposit {
+		miniumInitialCoin := MiniumInitialDepositRate.MulInt(coin.Amount).RoundInt()
+		miniumInitialDeposit = miniumInitialDeposit.Add(sdk.NewCoin(coin.Denom, miniumInitialCoin))
+	}
+
+	return
+}

app/app.go

@@ -202,7 +202,9 @@ func NewGaiaApp(
 				SignModeHandler: encodingConfig.TxConfig.SignModeHandler(),
 				SigGasConsumer:  ante.DefaultSigVerificationGasConsumer,
 			},
+			Codec:                app.appCodec,
 			IBCkeeper:            app.IBCKeeper,
+			GovKeeper:            &app.GovKeeper,
 			BypassMinFeeMsgTypes: bypassMinFeeMsgTypes,
 			GlobalFeeSubspace:    app.GetSubspace(globalfee.ModuleName),
 			StakingSubspace:      app.GetSubspace(stakingtypes.ModuleName),

docs/governance/proposal-types/community-pool-spend/proposal.json

@@ -11,7 +11,7 @@
   "deposit": [
     {
       "denom": "uatom",
-      "amount": "12000000"
+      "amount": "1000000000"
     }
   ]
 }

docs/governance/proposals/2020-10-blocks-per-year/proposal.json

@@ -11,7 +11,7 @@
   "deposit": [
     {
       "denom": "uatom",
-      "amount": "1000000"
+      "amount": "1000000000"
     }
   ]
 }

docs/governance/proposals/2020-11-inflation-rate-change/proposal.json

@@ -11,7 +11,7 @@
   "deposit": [
     {
       "denom": "uatom",
-      "amount": "1000000"
+      "amount": "1000000000"
     }
   ]
 }

docs/governance/proposals/2021-01-atom2021_marketing/proposal.json

@@ -11,7 +11,7 @@
   "deposit": [
     {
       "denom": "uatom",
-      "amount": "12000000"
+      "amount": "1000000000"
     }
   ]
 }

docs/governance/proposals/2021-04-lower-deposit-requirement/proposal.json

@@ -18,7 +18,7 @@
   "deposit": [
     {
       "denom": "uatom",
-      "amount": "1000000"
+      "amount": "1000000000"
     }
   ]
 }

docs/governance/proposals/2021-05-gravity-bridge-deployment/proposal.json

@@ -4,7 +4,7 @@
   "deposit": [
     {
       "denom": "uatom",
-      "amount": "1000000"
+      "amount": "1000000000"
     }
   ]
 }

docs/hub-tutorials/gaiad.md

@@ -631,7 +631,7 @@ Where `proposal.json` contains the following:
   "deposit": [
     {
       "denom": "stake",
-      "amount": "10000000"
+      "amount": "1000000000"
     }
   ]
 }

docs/ko/resources/gaiad.md

@@ -565,7 +565,7 @@ gaiad tx gov submit-proposal param-change <프로포절/파일/경로/프로포
   "deposit": [
     {
       "denom": "stake",
-      "amount": "10000000"
+      "amount": "1000000000"
     }
   ]
 }

docs/zh/resources/gaiad.md

@@ -543,7 +543,7 @@ gaiad tx gov submit-proposal param-change <path/to/proposal.json> \
   "deposit": [
     {
       "denom": "stake",
-      "amount": "10000000"
+      "amount": "1000000000"
     }
   ]
 }

tests/e2e/e2e_setup_test.go

@@ -75,7 +75,7 @@ var (
 	stakingAmountCoin = sdk.NewCoin(uatomDenom, stakingAmount)
 	tokenAmount       = sdk.NewCoin(uatomDenom, sdk.NewInt(3300000000)) // 3,300uatom
 	standardFees      = sdk.NewCoin(uatomDenom, sdk.NewInt(330000))     // 0.33uatom
-	depositAmount     = sdk.NewCoin(uatomDenom, sdk.NewInt(10000000))   // 10uatom
+	depositAmount     = sdk.NewCoin(uatomDenom, sdk.NewInt(1000000000)) // 1,000uatom
 	distModuleAddress = authtypes.NewModuleAddress(distrtypes.ModuleName).String()
 	proposalCounter   = 0
 )