diff --git a/.github/workflows/e2e-test-throttling.yml b/.github/workflows/e2e-test-throttling.yml index d5c1d0a6328..bff054e4cda 100644 --- a/.github/workflows/e2e-test-throttling.yml +++ b/.github/workflows/e2e-test-throttling.yml @@ -21,7 +21,7 @@ jobs: matrix: php: [ 8.1 ] db: [ pgsql ] - method: [ フロント画面ログイン_IP, フロント画面ログイン_会員, 管理画面ログイン_IP, 管理画面ログイン_会員, 会員登録, 問い合わせ, パスワード再発行, 注文確認_非会員購入, 注文確認_会員購入, 注文完了_非会員購入, 注文完了_会員購入, 会員情報編集, 配送先情報_追加, 配送先情報_編集, 配送先情報_削除, order_お届け先追加, order_お届け先変更 ] + method: [ フロント画面ログイン_IP, フロント画面ログイン_会員, 管理画面ログイン_IP, 管理画面ログイン_会員, 会員登録, 問い合わせ, パスワード再発行, 注文確認_非会員購入, 注文確認_会員購入, 注文完了_非会員購入, 注文完了_会員購入, 会員情報編集, 配送先情報_追加, 配送先情報_編集, 配送先情報_削除, order_お届け先追加, order_お届け先変更, 新規会員登録_入力 ] include: - db: pgsql database_url: postgres://postgres:password@127.0.0.1:5432/eccube_db diff --git a/app/config/eccube/packages/prod/eccube_rate_limiter.yaml b/app/config/eccube/packages/prod/eccube_rate_limiter.yaml index 2605faa3423..79f76a600b0 100644 --- a/app/config/eccube/packages/prod/eccube_rate_limiter.yaml +++ b/app/config/eccube/packages/prod/eccube_rate_limiter.yaml @@ -6,7 +6,13 @@ eccube: type: ip limit: 5 interval: '30 minutes' - entry: + entry: # 存在しているメールアドレスの確認を防止するためのスロットリング + route: entry + method: [ 'POST' ] + type: ip + limit: 25 + interval: '30 minutes' + entry_complete: # 不正な会員登録を防止するためのスロットリング route: entry method: [ 'POST' ] params: diff --git a/codeception/_support/Page/Front/EntryPage.php b/codeception/_support/Page/Front/EntryPage.php index acd3844a1e8..ad5a8a5fd06 100644 --- a/codeception/_support/Page/Front/EntryPage.php +++ b/codeception/_support/Page/Front/EntryPage.php @@ -15,11 +15,20 @@ class EntryPage extends AbstractFrontPage { + + private $formData = []; + public function __construct(\AcceptanceTester $I) { parent::__construct($I); } + /** + * @param $I + * @param $id + * + * @return EntryPage + */ public static function go($I) { $page = new self($I); @@ -28,11 +37,11 @@ public static function go($I) return $page; } - public function 新規会員登録($form = []) + public function フォーム入力($form = []) { $this->tester->amOnPage('/entry'); $email = uniqid().microtime(true).'@example.com'; - + $form += [ 'entry[name][name01]' => '姓', 'entry[name][name02]' => '名', @@ -49,10 +58,20 @@ public function 新規会員登録($form = []) 'entry[plain_password][second]' => 'password1234', 'entry[user_policy_check]' => '1', ]; - $this->tester->submitForm(['css' => '.ec-layoutRole__main form'], $form, ['css' => 'button.ec-blockBtn--action']); - $this->tester->seeInField(['id' => 'entry_email_first'], $form['entry[email][first]']); - $this->tester->click('.ec-registerRole form button.ec-blockBtn--action'); + $this->formData = $form; + return $this; + } + + public function 同意する() + { + $this->tester->submitForm(['css' => '.ec-layoutRole__main form'], $this->formData, ['css' => 'button.ec-blockBtn--action']); + $this->tester->seeInField(['id' => 'entry_email_first'], $this->formData['entry[email][first]']); + return $this; + } + public function 登録する() + { + $this->tester->click('.ec-registerRole form button.ec-blockBtn--action'); return $this; } } diff --git a/codeception/acceptance/EA07BasicinfoCest.php b/codeception/acceptance/EA07BasicinfoCest.php index a2361f6daa6..3bde8fc50a8 100644 --- a/codeception/acceptance/EA07BasicinfoCest.php +++ b/codeception/acceptance/EA07BasicinfoCest.php @@ -98,10 +98,13 @@ public function basicinfo_会員設定_仮会員機能(AcceptanceTester $I) // 会員登録 $faker = Fixtures::get('faker'); $email = microtime(true).'.'.$faker->safeEmail; - EntryPage::go($I)->新規会員登録([ - 'entry[email][first]' => $email, - 'entry[email][second]' => $email, - ]); + EntryPage::go($I) + ->フォーム入力([ + 'entry[email][first]' => $email, + 'entry[email][second]' => $email, + ]) + ->同意する() + ->登録する(); // 会員ステータスのチェック $page = CustomerManagePage::go($I); @@ -119,10 +122,13 @@ public function basicinfo_会員設定_仮会員機能(AcceptanceTester $I) // 会員登録 $I->logoutAsMember(); $email = microtime(true).'.'.$faker->safeEmail; - EntryPage::go($I)->新規会員登録([ - 'entry[email][first]' => $email, - 'entry[email][second]' => $email, - ]); + EntryPage::go($I) + ->フォーム入力([ + 'entry[email][first]' => $email, + 'entry[email][second]' => $email, + ]) + ->同意する() + ->登録する(); $I->logoutAsMember(); // 会員ステータスのチェック diff --git a/codeception/acceptance/EF09ThrottlingCest.php b/codeception/acceptance/EF09ThrottlingCest.php index 043ed34df9c..85bd15cc4f8 100644 --- a/codeception/acceptance/EF09ThrottlingCest.php +++ b/codeception/acceptance/EF09ThrottlingCest.php @@ -24,6 +24,8 @@ use Page\Front\ShoppingLoginPage; use Page\Front\ShoppingNonmemberPage; use Page\Front\ShoppingPage; +use Page\Front\EntryPage; + /** * @group throttling @@ -138,14 +140,18 @@ public function 会員登録(AcceptanceTester $I) for ($i = 0; $i < 5; $i++) { $I->expect('会員登録を行います：'.$i); - \Page\Front\EntryPage::go($I) - ->新規会員登録(); + EntryPage::go($I) + ->フォーム入力() + ->同意する() + ->登録する(); $I->see('現在、仮会員の状態です。', 'p.ec-reportDescription'); } $I->expect('試行回数上限を超過します'); - \Page\Front\EntryPage::go($I) - ->新規会員登録(); + EntryPage::go($I) + ->フォーム入力() + ->同意する() + ->登録する(); $I->see('試行回数の上限を超過しました。しばらくお待ちいただき、再度お試しください。', 'p.ec-reportDescription'); } @@ -748,4 +754,22 @@ public function order_お届け先変更(AcceptanceTester $I) $I->wait(1); $I->see('試行回数の上限を超過しました。しばらくお待ちいただき、再度お試しください。', 'p.ec-reportDescription'); } + + public function 新規会員登録_入力(AcceptanceTester $I) + { + $I->wantTo('EF0901-UC01-T18_会員登録_入力'); + + for ($i = 0; $i < 25; $i++) { + $I->expect('会員登録を行います：'.$i); + EntryPage::go($I) + ->フォーム入力() + ->登録する(); + } + + EntryPage::go($I) + ->フォーム入力() + ->登録する(); + $I->see('試行回数の上限を超過しました。しばらくお待ちいただき、再度お試しください。', 'p.ec-reportDescription'); + } + }