search

gBenkyous / inTime

3 stars 0 forks source link

サーバ側 - ログの導入②（最新ブランチに対して）_ログローテ現状 #89

Open gonzaresu-sengoku opened 7 months ago

gonzaresu-sengoku commented 7 months ago

ログローテに関して、アプリ起動時に1度だけファイルを開く処理が可能そうだったため一度試しで動かしてみました！ 現実装ですと以下の場合にファイル作成して開くような感じになってます。 ・ファイルが存在しない場合 ・ファイル名の日付と現日付が違う場合

今見つかっているバグとして、 ・アプリ起動後、ファイルがなくなった場合にエラー ・1度ファイルが作られた状態でアプリが再起動した際にエラー です。。。ロジックの問題だと思うので調査しながら修正します！

gonzaresu-sengoku commented 7 months ago

https://qiita.com/doranko/items/f8c7f1890d385fb4670c ⇒東さんのやつはまだ試せてないです！

doranko commented 7 months ago

@gonzaresu-sengoku

・アプリ起動後、ファイルがなくなった場合にエラー ・1度ファイルが作られた状態でアプリが再起動した際にエラー

これらですが「setupLogger が呼ばれた日時のログファイルが存在しない場合、一度だけロガーの設定が行われる」となっているからだと思います。

・アプリ起動後、ファイルがなくなった場合にエラー

-> ファイルが存在しないが、既に一度ロガーの設定が行われているので if 文内部が実行されず、存在しないファイルを参照する logger が返されていると思われる

・1度ファイルが作られた状態でアプリが再起動した際にエラー

-> ファイルが存在しているのでロガーの設定が行われず、未設定の logger が返されていると思われる

コードの該当箇所にもレビューコメントで記載しています。 ご連絡いただければ口頭での説明も実施しますのでお気軽にどうぞ。