新規会員登録（入力）画面のスロットリング追加

[KenTanaka]

概要(Overview・Refs Issue)

#5279

方針(Policy)

• メアド重複のみのスロットリングが実装できない為、新規会員登録（入力）画面のPOSTでスロットリングする
• スロットリング対象はIPのみとする（未ログイン状態である為）

実装に関する補足(Appendix)

テスト（Test)

• 手動での試験実施済み
  • 新規会員登録画面で閾値分の妥当性エラーとなるPOSTを繰り返す
  • 新規会員登録画面 <> 確認画面を往復する
  • 新規会員登録ができることを確認する
• E2Eテストを追加

相談（Discussion）

• メアド重複のみハンドリングしている訳ではない事に留意

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

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

レビュワー確認項目

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

[codecov-commenter]

Codecov Report

Merging #6038 (205f580) into 4.2 (ac096e5) will not change coverage.
Report is 10 commits behind head on 4.2.
The diff coverage is n/a.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@            Coverage Diff            @@
##                4.2    #6038   +/-   ##
=========================================
  Coverage     82.55%   82.55%           
  Complexity     6425     6425           
=========================================
  Files           475      475           
  Lines         25859    25859           
=========================================
  Hits          21348    21348           
  Misses         4511     4511           

Flag	Coverage Δ
E2E	69.46% <ø> (-0.01%)	⬇️
Unit	79.74% <ø> (+<0.01%)	⬆️

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

see 2 files with indirect coverage changes

[chihiro-adachi]

doc4も更新お願いしてよいでしょうか？
https://doc4.ec-cube.net/customize_throttling

[dotani1111]

@KenTanaka
入力画面にてスロットリングが掛かるE2Eテストをお願いします。

[KenTanaka]

@dotani1111
E2Eの方追加しました。
（EF0901-UC01-T18_会員登録_入力）

[chihiro-adachi]

マイページの会員編集画面でも同様のスロットリングが必要かと思いましたが、本PRでは対象外でしょうか？

[ji-eunsoo]

マイページの会員編集画面でも同様のスロットリングが必要かと思いましたが、本PRでは対象外でしょうか？

@chihiro-adachi
ありがとうございます。マイページの会員編集画面には既にスロットリングが入っており、対応不要となります。

ec-cube/app/config/eccube/packages/prod/eccube_rate_limiter.yaml

Line 17 in f8095c6

mypage_change:

[ji-eunsoo]

@chihiro-adachi
25回の入力テストで、p.ec-errorMessageの要素の内容を確認するように修正したため、
お手数をおかしますが、再度のほど、よろしくお願いいたします。

[chihiro-adachi]

マイページの会員編集画面でも同様のスロットリングが必要かと思いましたが、本PRでは対象外でしょうか？

@chihiro-adachi ありがとうございます。マイページの会員編集画面には既にスロットリングが入っており、対応不要となります。

ec-cube/app/config/eccube/packages/prod/eccube_rate_limiter.yaml

Line 17 in f8095c6

mypage_change:

ありがとうございます。了解です。

[chihiro-adachi]

@chihiro-adachi 25回の入力テストで、p.ec-errorMessageの要素の内容を確認するように修正したため、 お手数をおかしますが、再度のほど、よろしくお願いいたします。

問題ないです。