-
Notifications
You must be signed in to change notification settings - Fork 710
ZAPスキャン: /mypage/withdraw のスクリプト追加 #6674
Copy link
Copy link
Open
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保護
必要な作業
zap/scripts/mypage_withdraw.zstの作成- ログイン →
/mypage/withdrawGET →mode=confirmPOST →mode=completePOST のシーケンス before_script: admin_create_customer.zstで事前に会員を作成
- ログイン →
.github/workflows/zaproxy.ymlのmatrixにmypage_withdrawターゲットを追加
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels