Skip to content

Revert "イメージの取得先を清隆さんのリポジトリからEC-CUBEへ変更。"#6609

Merged
dotani1111 merged 1 commit into4.3from
revert-6533-feature/fix_zap
Jan 28, 2026
Merged

Revert "イメージの取得先を清隆さんのリポジトリからEC-CUBEへ変更。"#6609
dotani1111 merged 1 commit into4.3from
revert-6533-feature/fix_zap

Conversation

@zeniya0000
Copy link
Contributor

@zeniya0000 zeniya0000 commented Jan 28, 2026

Reverts #6533

Summary by CodeRabbit

  • Chores
    • セキュリティスキャンツール(OWASP ZAP)のCI/CD環境設定を更新しました。スキャンプロセスの実行と監視を改善し、より安定したセキュリティ検査環境を実現しています。

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Jan 28, 2026

📝 Walkthrough

Walkthrough

Docker Compose OWASP ZAP CI設定ファイルのZapプロキシイメージをバージョン更新し、コンテナのエントリーポイントコマンドを変更しました。構造的な変更はありません。

Changes

Cohort / File(s) 変更内容
OWASP ZAP設定更新
docker-compose.owaspzap.ci.yml
Zapプロキシイメージを ghcr.io/ec-cube/zaproxy:0.1.0 から kiy0taka/zap2docker-eccube に変更。エントリーポイントコマンドを sh -c "zap.sh ... && tail -f /dev/null" から bash -c "zap.sh ... && sleep infinity" に変更。

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 ゼップの新しい姿、イメージ更新
スリープモードで永遠に待つ
コンテナは眠らず、仕事をこなす
シンプルな変更、大きな意味
ザップよ、進め!新しい道へ ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed Pull request title clearly indicates reverting a previous commit that changed the Docker image source, which directly matches the changeset that reverts the Zap Docker image back to kiy0taka/zap2docker-eccube.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Comment @coderabbitai help to get the list of available commands and usage tips.

@dotani1111 dotani1111 merged commit 06a410b into 4.3 Jan 28, 2026
101 of 102 checks passed
@dotani1111 dotani1111 deleted the revert-6533-feature/fix_zap branch January 28, 2026 07:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants