Skip to content

ログパラメータの整理 #55

@meian

Description

@meian

ログ出力のパラメータを以下のように統一する。

引数パラメータを別に分離する

例えば、

logger := logs.FromContext(cmd.Context()).With("contestID", contestID)
は引数を直接渡してるが、これらを args 配下で出力するように変更する。

atgo/cmd/task.go

Lines 25 to 29 in 5e0d643

var taskID string
if len(args) > 0 {
taskID = args[0]
}
のように出力していない箇所は明示的に出力する。

下層レイヤーで自動解決した値を明示的にパラメータに出力する

logger = logger.With("contestID", info.ContestID).With("taskID", info.TaskID)
のように内部で解決されたパラメータで出力していない箇所があれば明示的にパラメータを出力する。

かつ、 TaskInfo などの構造体は構造体のままで出力する。
※構造体のままで出力できるかどうかを確認してから施行する

Metadata

Metadata

Assignees

Labels

improvementSome improvement app quality, operation, etc.

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions