Put each security issue in context block in Slack message
What does this MR do and why?
Describe in detail what your merge request does and why.
Putting each line into a separate context
block avoids the problem of too much text in one block causing the message to
cross the 3000 character block length limit.
This MR ensures that text added into a Slack message by Security::IssuesResult
is within the 3000 character limit (Slack docs).
This ensures that the Slack message that is generated when /chatops run release merge --security
is executed does not get rejected by Slack API.
Why?
Slack "section" block text can have a max of 3k chars. If the message is longer, the rest can go into another "section" block. Slack responds with 400 Bad Request if the limit is crossed. See gitlab-com/gl-infra/delivery#2145 (closed).
gitlab-com/gl-infra/delivery#2145 (closed)
Author Check-list
-
Has documentation been updated?