Skip to content

Draft: Specify more SAST analyzers for sort order

rossfuhrman requested to merge rf-order-more-analyzers into master

What does this MR do and why?

These SAST analyzers also have coverage from Semgrep. We need to give them a sort order for deduplication purposes.

Request access to see an example of this bug: https://gitlab.com/rossfuhrman/more-testing/-/security/vulnerability_report https://gitlab.com/rossfuhrman/more-testing/-/security/vulnerability_report

Steps to reproduce current bug:

  1. Merge MR that has Python and Go vulnerabilities and has the bandit and gosec analyzers run, but excludes the semgrep analyzer.
  2. Note that vulnerabilities show for both languages.
  3. Merge MR that turns on semgrep, so that it runs alongside bandit and gosec.
  4. Note that the bandit vulnerabilities are not duplicated, but the gosec vulnerabilities are.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by rossfuhrman

Merge request reports