Skip to content

Rename security dashboard graphql error alert

What does this MR do and why?

Related #396270 (closed)

This renames the error message shown in the alert one gets when an error occurs with the graphql api call to be less confusing and more generic. Original discussion and reasoning can be found in the issue #396270 (closed).

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
Screenshot_2024-02-15_at_15.50.27 Screenshot_2024-02-15_at_15.50.05

How to set up and validate locally

Prerequisites

  1. You need an EE license
  2. You need to have runners enabled (See $2408961 for setting up a runner)
  3. Import https://gitlab.com/gitlab-examples/security/security-reports
  4. Run a pipeline on master

Validate

  1. go to http://gdk.test:3000/-/security/dashboard/settings and make sure the security reports project is added
  2. go to http://gdk.test:3000/-/security/dashboard
  3. in dev tools > network, go to any graphql api call, right-click and select block url
  4. refresh page and validate that the new error message is shown
Edited by Lorenz van Herwaarden

Merge request reports