Show dismissal reason in vulnerability report table
Proposal
Show the dismissal reason (if any) for dismissed vulnerabilities in the table on the vulnerability report page in a neutral small badge:
Implementation steps
-
Include dismissalReason
inproject_vulnerabilities.query.graphql
,group_vulnerabilities.query.graphql
, orinstance_vulnerabilities.query.graphql
-
Adapt the template for status in ee/app/assets/javascripts/security_dashboard/components/shared/vulnerability_report/vulnerability_list.vue
so it passes state and dismissal reason to a new componentVulnerabilityListStatus
-
Create new component VulnerabilityListStatus
to render the status cell -
guard this behind the feature flag expose_dismissal_reason
-> does not need to be done in FE because BE simply does not pass alongdismissalReason
if ff is turned off. -
Adapt ee/spec/frontend/security_dashboard/components/shared/vulnerability_report/vulnerability_list_spec.js
spec for dismissal reasons in status -
Use GLBadge (small) in neutral to display dismissal reason
Verification steps
- Go to this verification project's vulnerability report
- Select dismissed status from status filter and uncheck other statuses
- Verify that dismissed vulnerabilities have a dismissal reason badge in the table
You can filter again on another status and dismiss a vulnerability/vulnerabilities with reason and verify whether this is reflected in the table too.
Edited by Lorenz van Herwaarden