Skip to content

Move vulnerability_list and project_vuln_app

Alexander Turinske requested to merge move-vulnerability-list-code into master

What does this MR do?

Move vulnerability_list, project_vuln_app, and related GraphQL quries

  • move project_vulnerabilities_app to the directory where the group/instance vulnerability code is, so that all of the security dashboard code is grouped together
  • move vulnerability_list to the directory that uses it
  • update files referencing them
  • move GraphQL queries related to vulnerabilities to ee/security_dashboard, where they are used

How to reproduce

https://gitlab.com/snippets/1956676

Screenshots

I renamed the project-level app file, so the project-level screenshots show that page continuing to work

I moved the vulnerability_list file, which is used by the project/group/instance-level dashboards, so all screenshots show all instances of the vulnerability_list working

Everything should look the same as before

Type Before (staging) After (local)
Project-level project_before project_after
Group-level group_before group_after
Instance-level instance_before instance_after

How to reproduce

https://gitlab.com/snippets/1956676

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
Edited by Alexander Turinske

Merge request reports