From 02b1b0089e2a4b59258b313c8ced72d88c2a8f82 Mon Sep 17 00:00:00 2001 From: Taeguk Kwon Date: Mon, 27 Nov 2023 18:00:22 +0700 Subject: [PATCH] refactor(gov): consistent logger usage in `EndBlocker()` (#18560) --- x/gov/abci.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/x/gov/abci.go b/x/gov/abci.go index 6f813552e3..79a69402b7 100644 --- a/x/gov/abci.go +++ b/x/gov/abci.go @@ -20,7 +20,7 @@ import ( func EndBlocker(ctx sdk.Context, keeper *keeper.Keeper) error { defer telemetry.ModuleMeasureSince(types.ModuleName, time.Now(), telemetry.MetricKeyEndBlocker) - logger := ctx.Logger().With("module", "x/"+types.ModuleName) + logger := keeper.Logger(ctx) // delete dead proposals from store and returns theirs deposits. // A proposal is dead when it's inactive and didn't get enough deposit on time to get into voting phase. rng := collections.NewPrefixUntilPairRange[time.Time, uint64](ctx.HeaderInfo().Time) @@ -70,7 +70,7 @@ func EndBlocker(ctx sdk.Context, keeper *keeper.Keeper) error { if err == nil { // purposely ignoring the error here not to halt the chain if the hook fails writeCache() } else { - keeper.Logger(ctx).Error("failed to execute AfterProposalFailedMinDeposit hook", "error", err) + logger.Error("failed to execute AfterProposalFailedMinDeposit hook", "error", err) } ctx.EventManager().EmitEvent( @@ -239,7 +239,7 @@ func EndBlocker(ctx sdk.Context, keeper *keeper.Keeper) error { if err == nil { // purposely ignoring the error here not to halt the chain if the hook fails writeCache() } else { - keeper.Logger(ctx).Error("failed to execute AfterProposalVotingPeriodEnded hook", "error", err) + logger.Error("failed to execute AfterProposalVotingPeriodEnded hook", "error", err) } logger.Info(