From 93f86fcd524a4f768b6dcb0a2c245dc14ff7ca41 Mon Sep 17 00:00:00 2001 From: mmsqe Date: Wed, 10 Jan 2024 23:58:46 +0800 Subject: [PATCH] fix: set packet seq in success log correctly (#5559) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit this bug was introduced in https://github.com/cosmos/ibc-go/commit/e93a467086d4b25f38632483ed6b2760867c0d2a Co-authored-by: DimitrisJim Co-authored-by: colin axnér <25233464+colin-axner@users.noreply.github.com> --- modules/apps/27-interchain-accounts/host/ibc_module.go | 2 +- modules/apps/transfer/ibc_module.go | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/modules/apps/27-interchain-accounts/host/ibc_module.go b/modules/apps/27-interchain-accounts/host/ibc_module.go index bbc68506fc4..c9d2ddcdc9d 100644 --- a/modules/apps/27-interchain-accounts/host/ibc_module.go +++ b/modules/apps/27-interchain-accounts/host/ibc_module.go @@ -128,7 +128,7 @@ func (im IBCModule) OnRecvPacket( ack = channeltypes.NewErrorAcknowledgement(err) logger.Error(fmt.Sprintf("%s sequence %d", err.Error(), packet.Sequence)) } else { - logger.Info("successfully handled packet sequence: %d", packet.Sequence) + logger.Info("successfully handled packet", "sequence", packet.Sequence) } // Emit an event indicating a successful or failed acknowledgement. diff --git a/modules/apps/transfer/ibc_module.go b/modules/apps/transfer/ibc_module.go index 02ed5fc9d61..2f0382d9e10 100644 --- a/modules/apps/transfer/ibc_module.go +++ b/modules/apps/transfer/ibc_module.go @@ -197,7 +197,7 @@ func (im IBCModule) OnRecvPacket( ackErr = err logger.Error(fmt.Sprintf("%s sequence %d", ackErr.Error(), packet.Sequence)) } else { - logger.Info("successfully handled ICS-20 packet sequence: %d", packet.Sequence) + logger.Info("successfully handled ICS-20 packet", "sequence", packet.Sequence) } }