Skip to content

Remove the AppSec ping from Custom SAST Rules bot message

Dominic Couture requested to merge dcouture-master-patch-55673 into master

What does this MR do and why?

The pings from this bot aren't always actionable by AppSec and the message itself should contain all the information necessary for software engineers to take action (at least after gitlab-com/gl-security/appsec/sast-custom-rules!26 is merged).

This removes the AppSec ping but leaves a message encouraging to ping whenever needed.

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

Before After

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

Merge request reports