easycorp/easy-log-handler を必須ライブラリに変更

[okazy]

概要(Overview・Refs Issue)

composer require easycorp/easy-log-handler を実施

背景

install 環境で easycorp/easy-log-handler を利用していたが、 composer.json では require-dev となっている状態となっていた。

ec-cube/app/config/eccube/packages/install/monolog.yml

Lines 16 to 21 in 124e613

	services:
	easycorp.easylog.handler:
	class: EasyCorp\EasyLog\EasyLogHandler
	public: false
	arguments:
	- '%kernel.logs_dir%/%kernel.environment%/site.log'

symfony/debug-pack を利用する上で easycorp/easy-log-handler が必須だったためインストール対象となり問題が起きていなかったが、 #4585 で Symfony のライブラリを更新したことにより必須ライブラリから外れてしまった。

ライブラリを composer install --no-dev でインストールした場合にwebインストーラでインストールしようとするとライブラリが読み込めずエラーとなっていた。

方針(Policy)

以下の２種類の対応が考えられるが、4.0.4以前の状態を正とし、前者の対応方針とした。

• [採用] easycorp/easy-log-handler を必須ライブラリとする
• install で easycorp/easy-log-handler を利用しない。

実装に関する補足(Appendix)

なし

テスト（Test)

以下のコマンドでライブラリを再インストールしてエラーが発生しなくなっていることを確認

composer install --no-scripts --no-dev --no-interaction --optimize-autoloader

相談（Discussion）

マイナーバージョン互換性保持のための制限事項チェックリスト

• 既存機能の仕様変更
• フックポイントの呼び出しタイミングの変更
• フックポイントのパラメータの削除・データ型の変更
• twigファイルに渡しているパラメータの削除・データ型の変更
• Serviceクラスの公開関数の、引数の削除・データ型の変更
• 入出力ファイル(CSVなど)のフォーマット変更

レビュワー確認項目

• 動作確認
• コードレビュー
• E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
• 互換性が保持されているか
• セキュリティ上の問題がないか