[ante] use infinite gas meter for transactions

Fixes #1549 (closed)

We do not use cosmos gas but our app still consumes gas and panics if the signer does not specify enough. Gas-wanted can be set to anything (it's fake, not used) and we've had issues in the past where we've needed to increase the recommended fake gas to avoid panics.

This wraps the ctx used for transaction processing with an infinite gas meter. It avoids out-of-gas panics in tx processing and also allows implementation changes to be made without consensus failure as long as the final state of the block is correct.

Note on regression tests: I set the gas wanted to 0 for tests. Previous version migrations which backdate their version then fail since they need gas. So I wired in an optional gas field for reg txs similar to sequence number. This works but if there's a better way, I'm all ears.

added 2 commits

• e4cf9df2 - 1 commit from branch develop
• 773ddcba - use infinite gas meter for transactions

Compare with previous version

marked this merge request as draft

changed milestone to %Release-1.115.0

approved this merge request

Superficial approve, obviously will need another one after test fixup.

Excellent artificial constraint to remove.

added 3 commits

• b47da0b8 - 1 commit from branch develop
• f2d614e5 - use infinite gas meter for transactions
• ebd8ad1d - reg test fixes

Compare with previous version

marked this merge request as ready

changed the description

added 1 commit

• bb597c0a - reg test fixes

Compare with previous version

approved this merge request

approved this merge request

added 3 commits

• 0e17b6fd - 1 commit from branch develop
• 75853a78 - use infinite gas meter for transactions
• d7f62124 - reg test fixes

Compare with previous version

enabled an automatic merge when the pipeline for d7f62124 succeeds

merged

mentioned in issue #1266 (closed)

mentioned in issue #1595

changed the description

added security label

mentioned in merge request !3204 (closed)