From 1a2197ec8e384a1fdccf675210e84282acd56b62 Mon Sep 17 00:00:00 2001 From: Arjit Chaudhary Date: Wed, 16 Oct 2019 03:01:46 +0530 Subject: [PATCH] Refactor "app/export.go" to use structured logging #12727 (#12729) * Refactor "app/export.go" to use structured logging #12727 * use mlog.String * use mlog.Bool, mlog.string needs 2 values? * Update export.go * s/reaction_UserId/user_id --- app/export.go | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/app/export.go b/app/export.go index 8ce7d50cdd..4836547002 100644 --- a/app/export.go +++ b/app/export.go @@ -5,7 +5,6 @@ package app import ( "encoding/json" - "fmt" "io" "net/http" "os" @@ -414,7 +413,7 @@ func (a *App) BuildPostReactions(postId string) (*[]ReactionImportData, *model.A user, err = a.Srv.Store.User().Get(reaction.UserId) if err != nil { if err.Id == store.MISSING_ACCOUNT_ERROR { // this is a valid case, the user that reacted might've been deleted by now - mlog.Info(fmt.Sprintf("Skipping reactions by user %v, since the entity doesn't exist anymore", reaction.UserId)) + mlog.Info("Skipping reactions by user since the entity doesn't exist anymore", mlog.String("user_id", reaction.UserId)) continue } return nil, err