ZAPスキャン: /mypage/withdraw のスクリプト追加
- Dominant language
- PHP
- Stars
- 788
- Forks
- 719
- Avg merge
- 4d 4h
- Merged PRs (30d)
- 39
Description
## 概要
ZAPセキュリティスキャンにおいて、退会処理(`/mypage/withdraw`)をカバーするZestスクリプトが存在しません。
## 背景
PR #6619 のレビュー時に発見しました。`WithdrawController` は `mode=confirm` → `mode=complete` で画面遷移しますが、対応する `mypage_withdraw.zst` がなく、CIのmatrixターゲットにも含まれていません。
## 対象コントローラー
- `src/Eccube/Controller/Mypage/WithdrawController.php`
- ルート: `/mypage/withdraw` (GET, POST)
- `mode=confirm`: 退会確認画面の表示
- `mode=complete`: 退会処理の実行(会員データ論理削除、メール送信、セッション破棄)
## セキュリティ上の懸念
退会処理には以下のセキュリティ上重要な処理が含まれるため、スキャン対象とすべきです。
- 会員データの論理削除
- メール送信
- セッション破棄
- CSRF保護
## 必要な作業
1. `zap/scripts/mypage_withdraw.zst` の作成
- ログイン → `/mypage/withdraw` GET → `mode=confirm` POST → `mode=complete` POST のシーケンス
- `before_script: admin_create_customer.zst` で事前に会員を作成
2. `.github/workflows/zaproxy.yml` のmatrixに `mypage_withdraw` ターゲットを追加
Contributor guide
Assessment
This issue has not been assessed yet.