Skip to content

Add detailed reasons when security issue is rejected

Reuben Pereira requested to merge rp/improve-implementation-issue-error-msgs into master

What does this MR do and why?

Describe in detail what your merge request does and why.

Add more detailed information in the comment posted when a security issue is not ready. These reasons are used in a few places, and providing more detail will help make them more understandable.

Here is an example of what the comment will look like with these changes: https://gitlab.com/gitlab-org/security/gitlab/-/issues/971#note_1567168236.

Since the reasons are now full sentences, the security issues table needed a slight modification to how it displays the reasons, to keep it understandable. The reasons will now be displayed as bullet points in the security issues table.

Here is an example of the modified security issues table: https://gitlab.com/gitlab-org/gitlab/-/issues/424529#note_1567213520.

gitlab-com/gl-infra/delivery#19658 (closed)

Author Check-list

  • Has documentation been updated?
Edited by Reuben Pereira

Merge request reports