Skip to content

Add Semgrep SAST job

rossfuhrman requested to merge rf-add-semgrep-sast into master

What does this MR do?

We are wrapping up Promote semgrep to general availability, and in the interest of dogfooding, would like to enable semgrep to run on the gitlab-org/gitlab project. It will run side-by-side with eslint for the time being.

The the semgrep job is taking quite a bit longer to run than the eslint job (7 minutes 17 seconds vs 1 minute 22 seconds), but we would like to proceed with enabling it so that we can establish a baseline to compare future improvements to semgrep.

Screenshots (strongly suggested)

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

Does this MR contain changes to processing or storing of credentials or tokens, authorization and authentication methods or other items described in the security review guidelines? If not, then delete this Security section.

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by rossfuhrman

Merge request reports