refactor(x/auth): use transaction service

julienrbrt · julienrbrt · commit 78a4c7a262c8 · 2024-04-05T17:53:27.000+02:00
diff --git a/core/transaction/service.go b/core/transaction/service.go
@@ -9,7 +9,7 @@ type ExecMode uint8
 // For backwards compatibility and easier casting, the exec mode values must be the same as in cosmos/cosmos-sdk/types package.
 const (
 	ExecModeCheck ExecMode = iota
-	_
+	ExecModeReCheck
 	ExecModeSimulate
 	_
 	_
diff --git a/types/context.go b/types/context.go
@@ -78,10 +78,10 @@ func (c Context) Logger() log.Logger                            { return c.logge
 func (c Context) VoteInfos() []abci.VoteInfo                    { return c.voteInfo }
 func (c Context) GasMeter() storetypes.GasMeter                 { return c.gasMeter }
 func (c Context) BlockGasMeter() storetypes.GasMeter            { return c.blockGasMeter }
-func (c Context) IsCheckTx() bool                               { return c.checkTx }   // Deprecated: use execMode instead
-func (c Context) IsReCheckTx() bool                             { return c.recheckTx } // Deprecated: use execMode instead
+func (c Context) IsCheckTx() bool                               { return c.checkTx }   // Deprecated: use core/transaction service instead
+func (c Context) IsReCheckTx() bool                             { return c.recheckTx } // Deprecated: use core/transaction service instead
 func (c Context) IsSigverifyTx() bool                           { return c.sigverifyTx }
-func (c Context) ExecMode() ExecMode                            { return c.execMode }
+func (c Context) ExecMode() ExecMode                            { return c.execMode } // Deprecated: use core/transaction service instead
 func (c Context) MinGasPrices() DecCoins                        { return c.minGasPrice }
 func (c Context) EventManager() EventManagerI                   { return c.eventManager }
 func (c Context) Priority() int64                               { return c.priority }
diff --git a/x/auth/CHANGELOG.md b/x/auth/CHANGELOG.md
@@ -33,6 +33,7 @@ Ref: https://keepachangelog.com/en/1.0.0/
 
 ### Improvements
 
+* []() Refactor ante handlers to use `transaction.Service` for getting exec mode.
 * [#18780](https://github.com/cosmos/cosmos-sdk/pull/18780) Move sig verification out of the for loop, into the authenticate method.
 * [#19188](https://github.com/cosmos/cosmos-sdk/pull/19188) Remove creation of `BaseAccount` when sending a message to an account that does not exist. 
     * When signing a transaction with an account that has not been created accountnumber 0 must be used
diff --git a/x/auth/ante/ante.go b/x/auth/ante/ante.go
@@ -42,7 +42,7 @@ func NewAnteHandler(options HandlerOptions) (sdk.AnteHandler, error) {
 	anteDecorators := []sdk.AnteDecorator{
 		NewSetUpContextDecorator(), // outermost AnteDecorator. SetUpContext must be called first
 		NewExtensionOptionsDecorator(options.ExtensionOptionChecker),
-		NewValidateBasicDecorator(),
+		NewValidateBasicDecorator(options.AccountKeeper),
 		NewTxTimeoutHeightDecorator(),
 		NewValidateMemoDecorator(options.AccountKeeper),
 		NewConsumeGasForTxSizeDecorator(options.AccountKeeper),
diff --git a/x/auth/ante/basic.go b/x/auth/ante/basic.go
@@ -1,6 +1,7 @@
 package ante
 
 import (
+	"cosmossdk.io/core/transaction"
 	errorsmod "cosmossdk.io/errors"
 	storetypes "cosmossdk.io/store/types"
 	"cosmossdk.io/x/auth/migrations/legacytx"
@@ -18,15 +19,20 @@ import (
 // If ValidateBasic passes, decorator calls next AnteHandler in chain. Note,
 // ValidateBasicDecorator decorator will not get executed on ReCheckTx since it
 // is not dependent on application state.
-type ValidateBasicDecorator struct{}
+type ValidateBasicDecorator struct {
+	ak AccountKeeper
+}
 
-func NewValidateBasicDecorator() ValidateBasicDecorator {
-	return ValidateBasicDecorator{}
+func NewValidateBasicDecorator(ak AccountKeeper) ValidateBasicDecorator {
+	return ValidateBasicDecorator{
+		ak: ak,
+	}
 }
 
 func (vbd ValidateBasicDecorator) AnteHandle(ctx sdk.Context, tx sdk.Tx, _ bool, next sdk.AnteHandler) (sdk.Context, error) {
 	// no need to validate basic on recheck tx, call next antehandler
-	if ctx.ExecMode() == sdk.ExecModeReCheck {
+	txService := vbd.ak.Environment().TransactionService
+	if txService.ExecMode(ctx) == transaction.ExecModeReCheck {
 		return next(ctx, tx, false)
 	}
 
@@ -36,7 +42,7 @@ func (vbd ValidateBasicDecorator) AnteHandle(ctx sdk.Context, tx sdk.Tx, _ bool,
 		}
 	}
 
-	return next(ctx, tx, ctx.ExecMode() == sdk.ExecModeSimulate)
+	return next(ctx, tx, false)
 }
 
 // ValidateMemoDecorator will validate memo given the parameters passed in
@@ -69,7 +75,7 @@ func (vmd ValidateMemoDecorator) AnteHandle(ctx sdk.Context, tx sdk.Tx, _ bool,
 		}
 	}
 
-	return next(ctx, tx, ctx.ExecMode() == sdk.ExecModeSimulate)
+	return next(ctx, tx, false)
 }
 
 // ConsumeTxSizeGasDecorator will take in parameters and consume gas proportional
@@ -101,7 +107,8 @@ func (cgts ConsumeTxSizeGasDecorator) AnteHandle(ctx sdk.Context, tx sdk.Tx, _ b
 	ctx.GasMeter().ConsumeGas(params.TxSizeCostPerByte*storetypes.Gas(len(ctx.TxBytes())), "txSize")
 
 	// simulate gas cost for signatures in simulate mode
-	if ctx.ExecMode() == sdk.ExecModeSimulate {
+	txService := cgts.ak.Environment().TransactionService
+	if txService.ExecMode(ctx) == transaction.ExecModeSimulate {
 		// in simulate mode, each element should be a nil signature
 		sigs, err := sigTx.GetSignaturesV2()
 		if err != nil {
@@ -143,7 +150,7 @@ func (cgts ConsumeTxSizeGasDecorator) AnteHandle(ctx sdk.Context, tx sdk.Tx, _ b
 		}
 	}
 
-	return next(ctx, tx, ctx.ExecMode() == sdk.ExecModeSimulate)
+	return next(ctx, tx, false)
 }
 
 // isIncompleteSignature tests whether SignatureData is fully filled in for simulation purposes
@@ -206,5 +213,5 @@ func (txh TxTimeoutHeightDecorator) AnteHandle(ctx sdk.Context, tx sdk.Tx, _ boo
 		)
 	}
 
-	return next(ctx, tx, ctx.ExecMode() == sdk.ExecModeSimulate)
+	return next(ctx, tx, false)
 }
diff --git a/x/auth/ante/basic_test.go b/x/auth/ante/basic_test.go
@@ -36,7 +36,7 @@ func TestValidateBasic(t *testing.T) {
 	invalidTx, err := suite.CreateTestTx(suite.ctx, privs, accNums, accSeqs, suite.ctx.ChainID(), signing.SignMode_SIGN_MODE_DIRECT)
 	require.NoError(t, err)
 
-	vbd := ante.NewValidateBasicDecorator()
+	vbd := ante.NewValidateBasicDecorator(suite.accountKeeper)
 	antehandler := sdk.ChainAnteDecorators(vbd)
 	_, err = antehandler(suite.ctx, invalidTx, false)
 
diff --git a/x/auth/ante/expected_keepers.go b/x/auth/ante/expected_keepers.go
@@ -4,20 +4,26 @@ import (
 	"context"
 
 	"cosmossdk.io/core/address"
+	"cosmossdk.io/core/appmodule"
 	"cosmossdk.io/x/auth/types"
 
 	sdk "github.com/cosmos/cosmos-sdk/types"
 )
 
+type HasEnvironment interface {
+	Environment() appmodule.Environment
+}
+
 // AccountKeeper defines the contract needed for AccountKeeper related APIs.
 // Interface provides support to use non-sdk AccountKeeper for AnteHandler's decorators.
 type AccountKeeper interface {
 	GetParams(ctx context.Context) (params types.Params)
 	GetAccount(ctx context.Context, addr sdk.AccAddress) sdk.AccountI
 	SetAccount(ctx context.Context, acc sdk.AccountI)
 	GetModuleAddress(moduleName string) sdk.AccAddress
-	AddressCodec() address.Codec
 	NewAccountWithAddress(ctx context.Context, addr sdk.AccAddress) sdk.AccountI
+	AddressCodec() address.Codec
+	Environment() appmodule.Environment
 }
 
 // FeegrantKeeper defines the expected feegrant keeper.
diff --git a/x/auth/ante/export_test.go b/x/auth/ante/export_test.go
@@ -0,0 +1,12 @@
+package ante
+
+import (
+	cryptotypes "github.com/cosmos/cosmos-sdk/crypto/types"
+	sdk "github.com/cosmos/cosmos-sdk/types"
+)
+
+var SimSecp256k1PubkeyInternal = simSecp256k1Pubkey
+
+func SetSVDPubKey(svd SigVerificationDecorator, ctx sdk.Context, acc sdk.AccountI, txPubKey cryptotypes.PubKey) error {
+	return svd.setPubKey(ctx, acc, txPubKey)
+}
diff --git a/x/auth/ante/ext.go b/x/auth/ante/ext.go
@@ -49,7 +49,7 @@ func (r RejectExtensionOptionsDecorator) AnteHandle(ctx sdk.Context, tx sdk.Tx,
 		return ctx, err
 	}
 
-	return next(ctx, tx, ctx.ExecMode() == sdk.ExecModeSimulate)
+	return next(ctx, tx, false)
 }
 
 func checkExtOpts(tx sdk.Tx, checker ExtensionOptionChecker) error {
diff --git a/x/auth/ante/fee.go b/x/auth/ante/fee.go
@@ -4,6 +4,7 @@ import (
 	"bytes"
 	"fmt"
 
+	"cosmossdk.io/core/transaction"
 	errorsmod "cosmossdk.io/errors"
 	"cosmossdk.io/x/auth/types"
 
@@ -45,7 +46,9 @@ func (dfd DeductFeeDecorator) AnteHandle(ctx sdk.Context, tx sdk.Tx, _ bool, nex
 		return ctx, errorsmod.Wrap(sdkerrors.ErrTxDecode, "Tx must be a FeeTx")
 	}
 
-	if ctx.ExecMode() != sdk.ExecModeSimulate && ctx.BlockHeight() > 0 && feeTx.GetGas() == 0 {
+	txService := dfd.accountKeeper.Environment().TransactionService
+	execMode := txService.ExecMode(ctx)
+	if execMode != transaction.ExecModeSimulate && ctx.BlockHeight() > 0 && feeTx.GetGas() == 0 {
 		return ctx, errorsmod.Wrap(sdkerrors.ErrInvalidGasLimit, "must provide positive gas")
 	}
 
@@ -55,7 +58,7 @@ func (dfd DeductFeeDecorator) AnteHandle(ctx sdk.Context, tx sdk.Tx, _ bool, nex
 	)
 
 	fee := feeTx.GetFee()
-	if ctx.ExecMode() != sdk.ExecModeSimulate {
+	if execMode != transaction.ExecModeSimulate {
 		fee, priority, err = dfd.txFeeChecker(ctx, tx)
 		if err != nil {
 			return ctx, err
@@ -67,7 +70,7 @@ func (dfd DeductFeeDecorator) AnteHandle(ctx sdk.Context, tx sdk.Tx, _ bool, nex
 
 	newCtx := ctx.WithPriority(priority)
 
-	return next(newCtx, tx, ctx.ExecMode() == sdk.ExecModeSimulate)
+	return next(newCtx, tx, false)
 }
 
 func (dfd DeductFeeDecorator) checkDeductFee(ctx sdk.Context, sdkTx sdk.Tx, fee sdk.Coins) error {
diff --git a/x/auth/ante/setup.go b/x/auth/ante/setup.go
@@ -67,14 +67,14 @@ func (sud SetUpContextDecorator) AnteHandle(ctx sdk.Context, tx sdk.Tx, _ bool,
 		}
 	}()
 
-	return next(newCtx, tx, ctx.ExecMode() == sdk.ExecModeSimulate)
+	return next(newCtx, tx, false)
 }
 
 // SetGasMeter returns a new context with a gas meter set from a given context.
 func SetGasMeter(ctx sdk.Context, gasLimit uint64) sdk.Context {
 	// In various cases such as simulation and during the genesis block, we do not
 	// meter any gas utilization.
-	if ctx.ExecMode() == sdk.ExecModeSimulate || ctx.BlockHeight() == 0 {
+	if ctx.ExecMode() == sdk.ExecModeSimulate || ctx.BlockHeight() == 0 { // NOTE: using environment here breaks the API of SetGasMeter, an alternative must be found for server/v2. ref: https://github.com/cosmos/cosmos-sdk/issues/19640
 		return ctx.WithGasMeter(storetypes.NewInfiniteGasMeter())
 	}
 
diff --git a/x/auth/ante/sigverify.go b/x/auth/ante/sigverify.go
@@ -10,6 +10,7 @@ import (
 	secp256k1dcrd "github.com/decred/dcrd/dcrec/secp256k1/v4"
 	"google.golang.org/protobuf/types/known/anypb"
 
+	"cosmossdk.io/core/transaction"
 	errorsmod "cosmossdk.io/errors"
 	storetypes "cosmossdk.io/store/types"
 	aa_interface_v1 "cosmossdk.io/x/accounts/interfaces/account_abstraction/v1"
@@ -215,7 +216,7 @@ func (svd SigVerificationDecorator) AnteHandle(ctx sdk.Context, tx sdk.Tx, _ boo
 
 	ctx.EventManager().EmitEvents(events)
 
-	return next(ctx, tx, ctx.ExecMode() == sdk.ExecModeSimulate)
+	return next(ctx, tx, false)
 }
 
 // authenticate the authentication of the TX for a specific tx signer.
@@ -280,7 +281,7 @@ func (svd SigVerificationDecorator) consumeSignatureGas(
 	pubKey cryptotypes.PubKey,
 	signature signing.SignatureV2,
 ) error {
-	if ctx.ExecMode() == sdk.ExecModeSimulate && pubKey == nil {
+	if svd.ak.Environment().TransactionService.ExecMode(ctx) == transaction.ExecModeSimulate && pubKey == nil {
 		pubKey = simSecp256k1Pubkey
 	}
 
@@ -310,7 +311,7 @@ func (svd SigVerificationDecorator) verifySig(ctx sdk.Context, tx sdk.Tx, acc sd
 	// we're in simulation mode, or in ReCheckTx, or context is not
 	// on sig verify tx, then we do not need to verify the signatures
 	// in the tx.
-	if ctx.ExecMode() == sdk.ExecModeSimulate || ctx.IsReCheckTx() || !ctx.IsSigverifyTx() {
+	if svd.ak.Environment().TransactionService.ExecMode(ctx) == transaction.ExecModeSimulate || ctx.IsReCheckTx() || !ctx.IsSigverifyTx() {
 		return nil
 	}
 
@@ -384,7 +385,7 @@ func (svd SigVerificationDecorator) setPubKey(ctx sdk.Context, acc sdk.AccountI,
 	if txPubKey == nil {
 		// if we're not in simulation mode, and we do not have a valid pubkey
 		// for this signer, then we simply error.
-		if ctx.ExecMode() != sdk.ExecModeSimulate {
+		if svd.ak.Environment().TransactionService.ExecMode(ctx) != transaction.ExecModeSimulate {
 			return fmt.Errorf("the account %s is without a pubkey and did not provide a pubkey in the tx to set it", acc.GetAddress().String())
 		}
 		// if we're in simulation mode, then we can populate the pubkey with the
@@ -479,7 +480,7 @@ func (vscd ValidateSigCountDecorator) AnteHandle(ctx sdk.Context, tx sdk.Tx, _ b
 		}
 	}
 
-	return next(ctx, tx, ctx.ExecMode() == sdk.ExecModeSimulate)
+	return next(ctx, tx, false)
 }
 
 // DefaultSigVerificationGasConsumer is the default implementation of SignatureVerificationGasConsumer. It consumes gas
diff --git a/x/auth/ante/sigverify_internal_test.go b/x/auth/ante/sigverify_internal_test.go
@@ -1,19 +1,41 @@
-package ante
+package ante_test
 
 import (
+	"context"
 	"testing"
 
 	"github.com/stretchr/testify/require"
 
+	"cosmossdk.io/core/appmodule"
+	"cosmossdk.io/core/transaction"
+	"cosmossdk.io/x/auth/ante"
 	authcodec "cosmossdk.io/x/auth/codec"
 	authtypes "cosmossdk.io/x/auth/types"
 
 	"github.com/cosmos/cosmos-sdk/crypto/keys/secp256k1"
 	sdk "github.com/cosmos/cosmos-sdk/types"
 )
 
+type mockAccount struct {
+	ante.AccountKeeper
+}
+
+func (*mockAccount) Environment() appmodule.Environment {
+	return appmodule.Environment{
+		TransactionService: &mockTransactionService{},
+	}
+}
+
+type mockTransactionService struct {
+	transaction.Service
+}
+
+func (*mockTransactionService) ExecMode(ctx context.Context) transaction.ExecMode {
+	return transaction.ExecMode(sdk.UnwrapSDKContext(ctx).ExecMode())
+}
+
 func TestSigVerify_setPubKey(t *testing.T) {
-	svd := SigVerificationDecorator{}
+	svd := ante.NewSigVerificationDecorator(&mockAccount{}, nil, nil, nil)
 
 	alicePk := secp256k1.GenPrivKey().PubKey()
 	bobPk := secp256k1.GenPrivKey().PubKey()
@@ -28,38 +50,38 @@ func TestSigVerify_setPubKey(t *testing.T) {
 	t.Run("on not sig verify tx - skip", func(t *testing.T) {
 		acc := &authtypes.BaseAccount{}
 		ctx = ctx.WithExecMode(sdk.ExecModeSimulate).WithIsSigverifyTx(false)
-		err := svd.setPubKey(ctx, acc, nil)
+		err := ante.SetSVDPubKey(svd, ctx, acc, nil)
 		require.NoError(t, err)
 	})
 
 	t.Run("on sim, populate with sim key, if pubkey is nil", func(t *testing.T) {
 		acc := &authtypes.BaseAccount{Address: aliceAddr}
 		ctx = ctx.WithExecMode(sdk.ExecModeSimulate).WithIsSigverifyTx(true)
-		err := svd.setPubKey(ctx, acc, nil)
+		err := ante.SetSVDPubKey(svd, ctx, acc, nil)
 		require.NoError(t, err)
-		require.Equal(t, acc.PubKey.GetCachedValue(), simSecp256k1Pubkey)
+		require.Equal(t, acc.PubKey.GetCachedValue(), ante.SimSecp256k1PubkeyInternal)
 	})
 
 	t.Run("on sim, populate with real pub key, if pubkey is not nil", func(t *testing.T) {
 		acc := &authtypes.BaseAccount{Address: aliceAddr}
 		ctx = ctx.WithExecMode(sdk.ExecModeSimulate).WithIsSigverifyTx(true)
-		err := svd.setPubKey(ctx, acc, alicePk)
+		err := ante.SetSVDPubKey(svd, ctx, acc, alicePk)
 		require.NoError(t, err)
 		require.Equal(t, acc.PubKey.GetCachedValue(), alicePk)
 	})
 
 	t.Run("not on sim, populate the address", func(t *testing.T) {
 		acc := &authtypes.BaseAccount{Address: aliceAddr}
 		ctx = ctx.WithExecMode(sdk.ExecModeFinalize).WithIsSigverifyTx(true)
-		err := svd.setPubKey(ctx, acc, alicePk)
+		err := ante.SetSVDPubKey(svd, ctx, acc, alicePk)
 		require.NoError(t, err)
 		require.Equal(t, acc.PubKey.GetCachedValue(), alicePk)
 	})
 
 	t.Run("not on sim, fail on invalid pubkey.address", func(t *testing.T) {
 		acc := &authtypes.BaseAccount{Address: aliceAddr}
 		ctx = ctx.WithExecMode(sdk.ExecModeFinalize).WithIsSigverifyTx(true)
-		err := svd.setPubKey(ctx, acc, bobPk)
+		err := ante.SetSVDPubKey(svd, ctx, acc, bobPk)
 		require.ErrorContains(t, err, "cannot be claimed")
 	})
 }
diff --git a/x/auth/ante/unordered.go b/x/auth/ante/unordered.go
diff --git a/x/auth/ante/unordered_test.go b/x/auth/ante/unordered_test.go
diff --git a/x/auth/ante/validator_tx_fee.go b/x/auth/ante/validator_tx_fee.go
diff --git a/x/auth/keeper/keeper.go b/x/auth/keeper/keeper.go

Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,7 @@ type ExecMode uint8`
`9`	`9`	`// For backwards compatibility and easier casting, the exec mode values must be the same as in cosmos/cosmos-sdk/types package.`
`10`	`10`	`const (`
`11`	`11`	`ExecModeCheck ExecMode = iota`
`12`		`- _`
	`12`	`+ ExecModeReCheck`
`13`	`13`	`ExecModeSimulate`
`14`	`14`	`_`
`15`	`15`	`_`
Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,7 @@`
`1`	`1`	`package ante`
`2`	`2`
`3`	`3`	`import (`
	`4`	`+ "cosmossdk.io/core/transaction"`
`4`	`5`	`errorsmod "cosmossdk.io/errors"`
`5`	`6`	`storetypes "cosmossdk.io/store/types"`
`6`	`7`	`"cosmossdk.io/x/auth/migrations/legacytx"`
`@@ -18,15 +19,20 @@ import (`
`18`	`19`	`// If ValidateBasic passes, decorator calls next AnteHandler in chain. Note,`
`19`	`20`	`// ValidateBasicDecorator decorator will not get executed on ReCheckTx since it`
`20`	`21`	`// is not dependent on application state.`
`21`		`-type ValidateBasicDecorator struct{}`
	`22`	`+type ValidateBasicDecorator struct {`
	`23`	`+ ak AccountKeeper`
	`24`	`+}`
`22`	`25`
`23`		`-func NewValidateBasicDecorator() ValidateBasicDecorator {`
`24`		`- return ValidateBasicDecorator{}`
	`26`	`+func NewValidateBasicDecorator(ak AccountKeeper) ValidateBasicDecorator {`
	`27`	`+ return ValidateBasicDecorator{`
	`28`	`+ ak: ak,`
	`29`	`+ }`
`25`	`30`	`}`
`26`	`31`
`27`	`32`	`func (vbd ValidateBasicDecorator) AnteHandle(ctx sdk.Context, tx sdk.Tx, _ bool, next sdk.AnteHandler) (sdk.Context, error) {`
`28`	`33`	`// no need to validate basic on recheck tx, call next antehandler`
`29`		`- if ctx.ExecMode() == sdk.ExecModeReCheck {`
	`34`	`+ txService := vbd.ak.Environment().TransactionService`
	`35`	`+ if txService.ExecMode(ctx) == transaction.ExecModeReCheck {`
`30`	`36`	`return next(ctx, tx, false)`
`31`	`37`	`}`
`32`	`38`
`@@ -36,7 +42,7 @@ func (vbd ValidateBasicDecorator) AnteHandle(ctx sdk.Context, tx sdk.Tx, _ bool,`
`36`	`42`	`}`
`37`	`43`	`}`
`38`	`44`
`39`		`- return next(ctx, tx, ctx.ExecMode() == sdk.ExecModeSimulate)`
	`45`	`+ return next(ctx, tx, false)`
`40`	`46`	`}`
`41`	`47`
`42`	`48`	`// ValidateMemoDecorator will validate memo given the parameters passed in`
`@@ -69,7 +75,7 @@ func (vmd ValidateMemoDecorator) AnteHandle(ctx sdk.Context, tx sdk.Tx, _ bool,`
`69`	`75`	`}`
`70`	`76`	`}`
`71`	`77`
`72`		`- return next(ctx, tx, ctx.ExecMode() == sdk.ExecModeSimulate)`
	`78`	`+ return next(ctx, tx, false)`
`73`	`79`	`}`
`74`	`80`
`75`	`81`	`// ConsumeTxSizeGasDecorator will take in parameters and consume gas proportional`
`@@ -101,7 +107,8 @@ func (cgts ConsumeTxSizeGasDecorator) AnteHandle(ctx sdk.Context, tx sdk.Tx, _ b`
`101`	`107`	`ctx.GasMeter().ConsumeGas(params.TxSizeCostPerByte*storetypes.Gas(len(ctx.TxBytes())), "txSize")`
`102`	`108`
`103`	`109`	`// simulate gas cost for signatures in simulate mode`
`104`		`- if ctx.ExecMode() == sdk.ExecModeSimulate {`
	`110`	`+ txService := cgts.ak.Environment().TransactionService`
	`111`	`+ if txService.ExecMode(ctx) == transaction.ExecModeSimulate {`
`105`	`112`	`// in simulate mode, each element should be a nil signature`
`106`	`113`	`sigs, err := sigTx.GetSignaturesV2()`
`107`	`114`	`if err != nil {`
`@@ -143,7 +150,7 @@ func (cgts ConsumeTxSizeGasDecorator) AnteHandle(ctx sdk.Context, tx sdk.Tx, _ b`
`143`	`150`	`}`
`144`	`151`	`}`
`145`	`152`
`146`		`- return next(ctx, tx, ctx.ExecMode() == sdk.ExecModeSimulate)`
	`153`	`+ return next(ctx, tx, false)`
`147`	`154`	`}`
`148`	`155`
`149`	`156`	`// isIncompleteSignature tests whether SignatureData is fully filled in for simulation purposes`
`@@ -206,5 +213,5 @@ func (txh TxTimeoutHeightDecorator) AnteHandle(ctx sdk.Context, tx sdk.Tx, _ boo`
`206`	`213`	`)`
`207`	`214`	`}`
`208`	`215`
`209`		`- return next(ctx, tx, ctx.ExecMode() == sdk.ExecModeSimulate)`
	`216`	`+ return next(ctx, tx, false)`
`210`	`217`	`}`
Original file line number	Diff line number	Diff line change
`@@ -49,7 +49,7 @@ func (r RejectExtensionOptionsDecorator) AnteHandle(ctx sdk.Context, tx sdk.Tx,`
`49`	`49`	`return ctx, err`
`50`	`50`	`}`
`51`	`51`
`52`		`- return next(ctx, tx, ctx.ExecMode() == sdk.ExecModeSimulate)`
	`52`	`+ return next(ctx, tx, false)`
`53`	`53`	`}`
`54`	`54`
`55`	`55`	`func checkExtOpts(tx sdk.Tx, checker ExtensionOptionChecker) error {`