[4.2] Add check invalid filename

[nanasess]

概要(Overview・Refs Issue)

closes #5743
ファイル管理機能のアップロード可能なファイル名を 英数字, 半角スペース, _-.() のみ許可する

方針(Policy)

 - ファイルアップロード時に正規表現 \A[a-zA-Z0-9_\-\.\(\) ]+\Z にてチェックする

実装に関する補足(Appendix)

テスト（Test)

ユニットテストを追加

相談（Discussion）

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

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

レビュワー確認項目

• 動作確認
• コードレビュー
• E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
• 互換性が保持されているか
• セキュリティ上の問題がないか
  • 権限を超えた操作が可能にならないか
  • 不要なファイルアップロードがないか
  • 外部へ公開されるファイルや機能の追加ではないか
  • テンプレートでのエスケープ漏れがないか

[chihiro-adachi]

@nanasess
Unit Testが落ちているようなのでご確認いただけますでしょうか。

[codecov-commenter]

Codecov Report

Merging #5760 (d0023c8) into 4.2 (0c42641) will increase coverage by 0.00%.
The diff coverage is 66.66%.

@@            Coverage Diff            @@
##                4.2    #5760   +/-   ##
=========================================
  Coverage     78.84%   78.85%           
- Complexity     6273     6275    +2     
=========================================
  Files           469      469           
  Lines         21058    21065    +7     
=========================================
+ Hits          16604    16611    +7     
  Misses         4454     4454           

Flag	Coverage Δ
E2E	64.85% <66.66%> (+0.01%)	⬆️
Unit	77.60% <72.72%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
...r/Admin/Setting/System/TwoFactorAuthController.php	0.00% <0.00%> (ø)
src/Eccube/Form/Type/Admin/CustomerType.php	95.74% <ø> (+2.12%)	⬆️
src/Eccube/Form/Type/RepeatedPasswordType.php	85.00% <ø> (ø)
src/Eccube/Twig/Extension/IntlExtension.php	82.60% <0.00%> (ø)
...Eccube/Controller/Admin/Content/FileController.php	84.58% <50.00%> (-0.28%)	⬇️
src/Eccube/Form/Type/Admin/MemberType.php	91.37% <85.71%> (+0.81%)	⬆️
.../Controller/Admin/Setting/System/LogController.php	74.41% <100.00%> (ø)
...be/Service/PurchaseFlow/Processor/TaxProcessor.php	82.45% <0.00%> (-1.76%)	⬇️
src/Eccube/Controller/Block/CalendarController.php	95.31% <0.00%> (+1.56%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[chihiro-adachi]

@nanasess
ありがとうございます。修正確認しました。