Add missing `anteHandlers` methods in `newLegacyCosmosAnteHandlerEip712`
omritoptix opened this issue · 0 comments
omritoptix commented
2 points here:
-
The
newLegacyCosmosAnteHandlerEip712
looks to handle Cosmos transactions, same asnewCosmosAnteHandler
with the only difference that legacy handler can handle transactions signed with Ledger. If they must be equivalent in everything else, probably the legacy handler should also useante.NewExtensionOptionsDecorator
anddelayedack.NewIBCProofHeightDecorator
(these two are used in "normal" handler but not in "legacy" one) -
please double-check that the
ante.NewExtensionOptionsDecorator
is intended to be configured like it is now. The checker is nil and, by looking at the config structure, someone could think that it allows all extensions but it forbids any of them