Skip to content

Add crashState to vulnerability details

David Pisek requested to merge 364793-add-location-crashstate-to-vulnerability-details into master

What does this MR do and why?

This MR is part of the pipeline's security tab's migration to GraphQL.

Going forward the VulnerabilityDetails component will be shared between the vulnerability detail's page (accessible via "Security & Compliance" -> "Vulnerability Report" -> Details for a given vulnerability) and the pipeline tab's vulnerability modal.

The modal currently renders location.crashState. This MR adds this to the VulnerabilityDetails component.

Screenshots or screen recordings

before after
Screen_Shot_2022-06-23_at_10.23.12_am Screen_Shot_2022-06-23_at_10.22.36_am

How to set up and validate locally

  1. Apply the following patch to add mock data: crash_state.patch
  2. Go to a project with vulnerabilities
  3. "Security & Compliance" -> "Vulnerability report" -> click on a vulnerability
  4. Verify that the crash state shows up
  5. Without the change that the patch introduces, it should not render

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #364793 (closed)

Edited by David Pisek

Merge request reports