Remove wasm times from our antehandler checkTx + recheckTx stack

osmosis-labs · May 24, 2024 · 0ed3f01 · 0ed3f01
1 parent 815d4bf
commit 0ed3f01
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/x/ibc-hooks/wasm_hook.go b/x/ibc-hooks/wasm_hook.go
@@ -298,6 +298,9 @@ func (h WasmHooks) OnAcknowledgementPacketOverride(im IBCMiddleware, ctx sdk.Con
 	if err != nil {
 		return err
 	}
+	if ctx.IsCheckTx() || ctx.IsReCheckTx() {
+		return nil
+	}
 
 	if !h.ProperlyConfigured() {
 		// Not configured. Return from the underlying implementation

diff --git a/x/ibc-rate-limit/ibc_module.go b/x/ibc-rate-limit/ibc_module.go
@@ -157,6 +157,9 @@ func (im *IBCModule) OnAcknowledgementPacket(
 	acknowledgement []byte,
 	relayer sdk.AccAddress,
 ) error {
+	if ctx.IsCheckTx() || ctx.IsReCheckTx() {
+		return im.app.OnAcknowledgementPacket(ctx, packet, acknowledgement, relayer)
+	}
 	var ack channeltypes.Acknowledgement
 	if err := json.Unmarshal(acknowledgement, &ack); err != nil {
 		return errorsmod.Wrapf(sdkerrors.ErrUnknownRequest, "cannot unmarshal ICS-20 transfer packet acknowledgement: %v", err)