Dismissal Types(FE): Add dismissal type reason to vuln details page head
Why are we doing this work
In order to show the exact reason why a particular dismissal type was chosen, we need to include this information in the head on the vulnerability details page. Currently we only show "Dismissed", this should be enhanced to include the dismissal type reason as illustrated in these mockups:
Localization of the dismissal type reason will need to be considered in the refinement.
Relevant links
Non-functional requirements
-
Documentation: -
Feature flag: dismissal_reason
-
Performance: -
Testing:
Implementation plan
-
frontend head: use dismissalReason
invulnerability
to show reason inStatusDescription
Verification steps
The feature flag dismissal_reason
has been enabled on lorenzvanherwaarden/security-reports
.
- Go to https://gitlab.com/lorenzvanherwaarden/security-reports/-/security/vulnerabilities/73661285 (this is a vulnerability which has been dismissed with a reason by manually doing the graphql vulnerabilityDismiss mutation)
- Verify that the status header shows: "Dismissed: Used in tests"
Another example
- Go to https://gitlab.com/lorenzvanherwaarden/security-reports/-/security/vulnerabilities/73661283
- Verify that the status header shows: "Dismissed: Mitigating control"
Edited by Lorenz van Herwaarden