ec-cube icon indicating copy to clipboard operation
ec-cube copied to clipboard

脆弱性チェックの自動化対応

Open sai-gillingham opened this issue 2 years ago • 1 comments

概要(Overview・Refs Issue)

以前の報告された脆弱性をまた発生しないように自動チェッカーを作りました。

方針(Policy)

実装に関する補足(Appendix)

テスト(Test)

参考のため、成功したgithub action テストを共有します。

https://github.com/sai-gillingham/ec-cube/actions/runs/2829818259

相談(Discussion)

マイナーバージョン互換性保持のための制限事項チェックリスト

  • [ ] 既存機能の仕様変更はありません
  • [ ] フックポイントの呼び出しタイミングの変更はありません
  • [ ] フックポイントのパラメータの削除・データ型の変更はありません
  • [ ] twigファイルに渡しているパラメータの削除・データ型の変更はありません
  • [ ] Serviceクラスの公開関数の、引数の削除・データ型の変更はありません
  • [ ] 入出力ファイル(CSVなど)のフォーマット変更はありません

レビュワー確認項目

  • [ ] 動作確認
  • [ ] コードレビュー
  • [ ] E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
  • [ ] 互換性が保持されているか
  • [ ] セキュリティ上の問題がないか
    • [ ] 権限を超えた操作が可能にならないか
    • [ ] 不要なファイルアップロードがないか
    • [ ] 外部へ公開されるファイルや機能の追加ではないか
    • [ ] テンプレートでのエスケープ漏れがないか

sai-gillingham avatar Aug 10 '22 11:08 sai-gillingham

Codecov Report

Merging #5523 (fe38b1c) into 4.2 (c3ecb92) will decrease coverage by 0.00%. The diff coverage is n/a.

@@             Coverage Diff              @@
##                4.2    #5523      +/-   ##
============================================
- Coverage     78.83%   78.82%   -0.01%     
  Complexity     6253     6253              
============================================
  Files           469      469              
  Lines         20992    20992              
============================================
- Hits          16548    16547       -1     
- Misses         4444     4445       +1     
Flag Coverage Δ
E2E 64.79% <ø> (+0.01%) :arrow_up:
Unit 77.52% <ø> (-0.01%) :arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...be/Service/PurchaseFlow/Processor/TaxProcessor.php 75.43% <0.00%> (-3.51%) :arrow_down:
...ontroller/Admin/Product/ProductClassController.php 90.85% <0.00%> (+0.57%) :arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

codecov-commenter avatar Aug 10 '22 11:08 codecov-commenter