EC-CUBE / EC-CUBE/ec-cube

ZAPスキャン: /mypage/withdraw のスクリプト追加

Open
#6,674 0 comments 0 reactions 1 assignee Claimed by @zeniya0000 View on GitHub
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

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.