Remove size information from the message of ExceedLimitError

What does this MR do?

The size limiter was already deployed into production and track mode was enabled (gitlab-com/gl-infra/production#4451 (closed) and gitlab-com/gl-infra/production#4487 (closed)).

After enabling the flags, Sentry observed a series of incoming events as expected. However, they are not grouped into one error group. Sentry tends to group the events based on their worker classes and messages. Having a dynamic parameter in the message prevents Sentry from grouping the events, even though the events are for the same worker class.

This is an example of the event message: WebHookWorker job exceeds payload size limit (163810/100000). In the payload sent to Sentry, the current size and limit was included. Hence, the size information is redundant here.

This MR is to simply remove that part.

Screenshots (strongly suggested)

Before

Screen_Shot_2021-05-17_at_17.46.20

After

Screen_Shot_2021-05-17_at_18.01.29

Screen_Shot_2021-05-17_at_18.01.43

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

Merge request reports

Loading