From e335a70f35b7c173d22af73f252f06372cfce7af Mon Sep 17 00:00:00 2001 From: Joe Bowman Date: Mon, 7 Mar 2022 13:23:01 +0000 Subject: [PATCH] call packet.GetSequence() rather than passing the func as argument (#995) (cherry picked from commit fc452ac42d06a0119d0035a524fe90dc8e4cd466) # Conflicts: # CHANGELOG.md --- CHANGELOG.md | 4 ++++ modules/core/04-channel/keeper/packet.go | 2 +- 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 960f63aa123..b3183ca3311 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -52,7 +52,11 @@ Ref: https://keepachangelog.com/en/1.0.0/ ### Bug Fixes * (client) [\#941](https://github.com/cosmos/ibc-go/pull/941) Classify client states without consensus states as expired +<<<<<<< HEAD * (transfer) [\#978](https://github.com/cosmos/ibc-go/pull/978) Support base denoms with slashes in denom validation +======= +* (modules/core/04-channel) [\#994](https://github.com/cosmos/ibc-go/pull/944) Call `packet.GetSequence()` rather than passing func in `AcknowledgePacket` log output +>>>>>>> fc452ac (call packet.GetSequence() rather than passing the func as argument (#995)) ## [v1.2.6](https://github.com/cosmos/ibc-go/releases/tag/v1.2.6) - 2022-02-03 diff --git a/modules/core/04-channel/keeper/packet.go b/modules/core/04-channel/keeper/packet.go index a31f511f074..d226e9818fd 100644 --- a/modules/core/04-channel/keeper/packet.go +++ b/modules/core/04-channel/keeper/packet.go @@ -488,7 +488,7 @@ func (k Keeper) AcknowledgePacket( // log that a packet has been acknowledged k.Logger(ctx).Info( "packet acknowledged", - "sequence", packet.GetSequence, + "sequence", packet.GetSequence(), "src_port", packet.GetSourcePort(), "src_channel", packet.GetSourceChannel(), "dst_port", packet.GetDestPort(),