fix: set packet seq in success log correctly (cosmos#5559)

this bug was introduced in cosmos@e93a467 Co-authored-by: DimitrisJim <[email protected]> Co-authored-by: colin axnér <[email protected]>
notional-labs · Jan 10, 2024 · 93f86fc · 93f86fc
1 parent d487fb8
commit 93f86fc
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git 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
@@ -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)
 		}
 	}