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

無駄なログ出力の抑制 #5082

Closed
tao-s opened this issue Jul 2, 2021 · 3 comments
Closed

無駄なログ出力の抑制 #5082

tao-s opened this issue Jul 2, 2021 · 3 comments
Labels
improvement 機能改善
Milestone

Comments

@tao-s
Copy link
Contributor

tao-s commented Jul 2, 2021

概要(Overview)

現在のデフォルトの設定では、site, admin, frontでログが出力されるが、prod環境であったとしてもそれぞれほぼ同じ様な内容が出力される。また、symfonyやcomposerのライブラリに関する Deprecatedや、 security.INFO ront.INFO など本番環境では必要無さそうなapacheのアクセスログみたいなものが大量に出力されて見辛いしログファイルがディスクを圧迫したりする。

期待する内容(Expect) or 要望 (Requirement)

prod環境ではERRORなどクリティカルなエラーが発生した場合にのみ詳細を出力する様にしたい
参考: https://symfony.com/doc/3.4/logging.html

環境 (environment)

  • EC-CUBE: 4.0.5

関連情報 (Ref)

ref #4668

@watarukashii
Copy link
Contributor

本件、私も気になって調べて改善策をご提案させていただきますので、ご検討頂くことは可能ですか。

検討内容

  1. Deprecated エラー
  2. https://github.com/EC-CUBE/ec-cube/blob/4.1/src/Eccube/EventListener/LogListener.php にて出力される「INIT」「PROCESS START」「LOGIC START」「LOGIC END」「PROCESS END」

修正内容

  1. Deprecated エラー

symfony4.1 系からは、error_reporting から除外対象はログに出力されなくなったということです。
ただ、symfony3.4 系では、monolog の修正が必要とのことで合わせて画像添付しております。

・./index.php にて error_reporting(E_ALL & ~E_DEPRECATED & ~E_USER_DEPRECATED);
・./app/config/eccube/packages/prod/monolog.yml の調整

SS-screenshot- 2022-02-25 12 52 08

SS-screenshot- 2022-02-25 12 52 24

参考URL
symfony/symfony#26267 (comment)

  1. https://github.com/EC-CUBE/ec-cube/blob/4.1/src/Eccube/EventListener/LogListener.php にて出力される「INIT」「PROCESS START」「LOGIC START」「LOGIC END」「PROCESS END」

ログレベルがinfo だったのと、本番モードは不要かと考えており info -> debug に変えるというのはいかがでしょうか?
あわせて、./app/config/eccube/packages/prod/monolog.yml の調整を行っております。

SS-screenshot- 2022-02-25 13 21 05

まとめ

上記の対応で、一番多かったDeprecatedエラーは本番モードでなくなり、スッキリとしたログが閲覧が可能となりました。

関連課題

#5082
#4668
#4775

@watarukashii
Copy link
Contributor

@tao-s こんにちは、本件、下記にてプルリクエストの承認をいただきました。
問題なさそうであればissue クローズの程よろしくおねがいします。
#5322

@chihiro-adachi chihiro-adachi modified the milestones: 4.1.x, 4.1.3 Apr 20, 2022
@chihiro-adachi
Copy link
Contributor

#5322 で対応のためクローズします

@Yangsin Yangsin modified the milestones: 4.1.3, 4.2.0 Apr 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement 機能改善
Projects
None yet
Development

No branches or pull requests

4 participants