Remove semgrep-sast CI rules

Dominic Couturerequested to merge
dcouture-remove-semgrep-ci into main

What does this MR do?

Follow up to Add GitLab Advanced SAST to CI/CD config (!4965 - merged)

In the default SAST CI template Semgrep does not run for languages where GitLab Advanced SAST is configured (source). Because of the rules override it ran anyway. This removes the custom rule.

Why was this MR needed?

The semgrep job fills the vulnerability report with false positives and isn't needed anymore because GitLab Advanced SAST is enabled here and covers this repository's programming language.

What's the best way to test this MR?

Run the pipelines

What are the relevant issue numbers?

Somewhat related to https://gitlab.com/gitlab-com/gl-security/product-security/appsec/appsec-team/-/issues/570#note_2227725324

Edited by Dominic Couture

Merge request reports