IBC Antehandler no longer working

[faddat]

Summary of Bug

https://www.mintscan.io/cosmos/txs/54989DC46695DD97314C4E5FACCC8234A255A8983D4456B0E2AEAAF9870DFB1C

When it was working, duplicate ibc transactions would be rejected but that is no longer happening.

Version

Gaia, Osmosis, Terra, Juno

Steps to Reproduce

screen -S gaia
git clone https://github.com/cosmos/gaia
cd gaia
bash contrib/statesync.bash

screen -S osmosis
git clone https://github.com/osmosis-labs/osmosis
cd osmosis
git checkout v6.2.0
bash scripts/statesync.sh

screen -S hermes
git clone https://github.com/notional-labs/notional
mkdir ~/.hermes
cp notional/relaying/hermes/osmo-gaia.toml ~/.hermes/config.toml
# add keys as instructed at https://github.com/notional-labs/notional/relaying/hermes
hermes start

You'll see that duplicate transactions aren't being rejected. 


____

#### For Admin Use

- [ ] Not duplicate issue
- [ ] Appropriate labels applied
- [ ] Appropriate contributors tagged/assigned

[crodriguezvega]

@faddat Thanks for opening this issue.

I seem to be able to reproduce the problem locally running our simapp, so we will investigate this further. I have attached for reference the files tx-relayer-1.txt and tx-relayer-2.txt with the same messages committed by two different relayers in the same block.

@colin-axner @AdityaSripal I noticed that in the app.go of our simapp the antehandler is instantiated using the NewAnteHandler from cosmos-sdk/x/auth/ante instead of the custom NewAnteHandler function that is used, for example, in gaia. I guess we need to do for our simapp something similar as gaia does?

[crodriguezvega]

Closed by #950