Skip to content

Commit

Permalink
fix: set packet seq in success log correctly (#5559)
Browse files Browse the repository at this point in the history
this bug was introduced in e93a467

Co-authored-by: DimitrisJim <d.f.hilliard@gmail.com>
Co-authored-by: colin axnér <25233464+colin-axner@users.noreply.github.com>
  • Loading branch information
3 people committed Jan 10, 2024
1 parent d487fb8 commit 93f86fc
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion modules/apps/27-interchain-accounts/host/ibc_module.go
Original file line number Diff line number Diff line change
Expand Up @@ -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.
Expand Down
2 changes: 1 addition & 1 deletion modules/apps/transfer/ibc_module.go
Original file line number Diff line number Diff line change
Expand Up @@ -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)
}
}

Expand Down

0 comments on commit 93f86fc

Please sign in to comment.