Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

serverのLog level修正 #77

Merged
merged 5 commits into from
Sep 10, 2024
Merged

serverのLog level修正 #77

merged 5 commits into from
Sep 10, 2024

Conversation

tetter27
Copy link
Collaborator

@tetter27 tetter27 commented Sep 9, 2024

  • 既存のログをINFO, DEBUG, TRACE, WARN, ERRORの5つへ分類
  • spanの範囲が正しく適用できていなかった問題を修正 (Connection, Session毎にログがグルーピングされるように修正)
  • README.mdのログレベル設定コマンドのtypoを修正

ログレベルの大まかなルールは以下の通り。

  • INFO: 基本的なイベントや状態変化を表示
  • TRACE: 各関数の実行/完了や通過を表示
  • DEBUG: メッセージの中身や細かいパラメータを表示
  • WARN: 終了には繋がらないが好ましくない動作を表示
  • ERROR: 終了につながるエラーを表示

@yuki-uchida yuki-uchida merged commit 0fe58b5 into master Sep 10, 2024
2 checks passed
@yuki-uchida yuki-uchida deleted the refactor/log_levels branch September 10, 2024 02:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants