Sending TX's multiple times in quick succession from the same sender account, fails

njmurarka · March 1, 2020, 1:58am

Hi. I thought that the “broadcast-mode” option for the CLI’s “tx” interface would ensure that I could “rapid-fire” transactions through from the same account, but I keep getting errors.

Here is the kind of command I am running:

echo mypassword | foocli tx bank send foo13m5p9y4fsp45utx98jxhjj9z8ul45v4a0mdz7j foo13m5p9y4fsp45utx98jxhjj9z8ul45v4a0mdz7j 24008abc --gas-prices 0.01abc --from validator --yes --broadcast-mode sync

The first such TX succeeds but anything sent quickly after this fails with the following subset of the JSON response:

‘raw_log’: ‘{“codespace”:“sdk”,“code”:4,“message”:“signature verification ’
'failed; verify correct account sequence and chain-id”}’,

I know the chain-id is correct. So I am guessing now that the account sequence # is not being incremented. But that does not make sense either to me – isn’t that sequence # going to increment when you are in sync mode?

Please advise.

njmurarka · March 2, 2020, 11:38pm

So as an update for others.

I was assuming here that if foocli is run in broadcast-mode’s “sync” mode, it would return AND the next time I ran foocli, it is guaranteed to have the next sequence number for that account.

I guess that’s not true. When it returns from sync mode, indeed it is safe to send another tx with that next sequence number, BUT you need to manually provide the incremented sequence number… foocli will not automatically get it (foocli only gets the sequence number reflective of the last block, not reflective of tx’s that are in the mempool).

It does mean of course, that you cannot have multiple entities sending transactions through the same account… they would have no way to now which sequence number to use.

NicolasMahe · March 4, 2020, 8:37am

Since cosmos-sdk v0.38 it is not possible to broadcast multiple transactions with the same account within the same block, even with an incremented account sequence.

The code responsible for this is the IncrementSequenceDecorator and its doc says:

As a result, this has the side effect that subsequent and sequential txs orginating from the same account cannot be handled correctly in a reliable way. To send sequential txs orginating from the same account, it is recommended to instead use multiple messages in a tx.

github.com

cosmos/cosmos-sdk/blob/274f389111c323c850c981c0de1b7b57eeb23912/x/auth/ante/sigverify.go#L219-L230


      
          // IncrementSequenceDecorator handles incrementing sequences of all signers.
          // Use the IncrementSequenceDecorator decorator to prevent replay attacks. Note,
          // there is no need to execute IncrementSequenceDecorator on CheckTx or RecheckTX
          // since it is merely updating the nonce. As a result, this has the side effect
          // that subsequent and sequential txs orginating from the same account cannot be
          // handled correctly in a reliable way. To send sequential txs orginating from the
          // same account, it is recommended to instead use multiple messages in a tx.
          //
          // CONTRACT: The tx must implement the SigVerifiableTx interface.
          type IncrementSequenceDecorator struct {
          	ak keeper.AccountKeeper
          }

As my project really needs to create multiple transactions from the same account in the same block, we came up with a hack to force IncrementSequenceDecorator to always behave as the transaction is not a CheckTx or RecheckTX by creating forceCheckTxProxyDecorator to wrap the original IncrementSequenceDecorator.
Here is how we use it in the app.go:

github.com

mesg-foundation/engine/blob/8554d14bda33b2d74b88bb6946f8c6e6834b4c99/app/app.go#L318-L339


      
          	// The AnteHandler handles signature verification and transaction pre-processing
          	app.SetAnteHandler(
          		sdk.ChainAnteDecorators(
          			ante.NewSetUpContextDecorator(), // outermost AnteDecorator. SetUpContext must be called first
          			ante.NewMempoolFeeDecorator(),
          			ante.NewValidateBasicDecorator(),
          			ante.NewValidateMemoDecorator(app.accountKeeper),
          			ante.NewConsumeGasForTxSizeDecorator(app.accountKeeper),
          			ante.NewSetPubKeyDecorator(app.accountKeeper), // SetPubKeyDecorator must be called before all signature verification decorators
          			ante.NewValidateSigCountDecorator(app.accountKeeper),
          			ante.NewDeductFeeDecorator(app.accountKeeper, app.supplyKeeper),
          			ante.NewSigGasConsumeDecorator(app.accountKeeper, auth.DefaultSigVerificationGasConsumer),
          			ante.NewSigVerificationDecorator(app.accountKeeper),
          			newForceCheckTxProxyDecorator(ante.NewIncrementSequenceDecorator(app.accountKeeper)), // innermost AnteDecorator
          		),
          		// Previous implementation:
          		// auth.NewAnteHandler(
          		// 	app.accountKeeper,
          		// 	app.supplyKeeper,
          		// 	auth.DefaultSigVerificationGasConsumer,

This file has been truncated. show original

Hope this help

Topic		Replies	Views
Multi Message Broadcast Struggle Cosmos-SDK	1	659	November 2, 2021
Same sequence from LCD after committing a block Tendermint	0	409	October 14, 2020
Help with "signature verification failed" Cosmos-SDK	4	6461	July 19, 2018
No transactions in block 15213800 Miscellaneous	2	201	December 27, 2023
Redelegation worked but with errors Cosmos-SDK	1	892	March 17, 2019

Sending TX's multiple times in quick succession from the same sender account, fails

Related Topics