Skip to content

Add new security charts page

Alexander Turinske requested to merge 227878-add-new-security-charts-page into master

What does this MR do?

Add new security charts page

  • create new group/security/vulnerabilities#index route
  • create new controller to serve up vulnerabilities page
  • create new nav link
  • remove now redundant charts on vulnerabilities page
  • update tests

Screenshots

Previous one page New Page 1 New Page 2 Unavailable Page
Screen_Shot_2020-07-30_at_4.07.23_PM Screen_Shot_2020-07-30_at_3.57.01_PM Screen_Shot_2020-07-30_at_3.57.10_PM Screen_Shot_2020-07-31_at_7.39.33_AM

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • 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

Related to #227878 (closed)

Edited by Alexander Turinske

Merge request reports