Add url to vulnerability list page
Add the URL to the vulnerability listing page, as per the design below:
Non-functional requirements
-
Documentation: No. Our docs screenshots don't currently show DAST results. -
Feature flag: Not needed -
Performance: -
Testing: Yes, add a new test for path
Implementation plan
-
Update the client-side GraphQL query vulnerability.fragment.graphql#L25 to include location data in DAST entries.
... on VulnerabilityLocationDast {
hostname
path
}
-
Update vulnerability_list.vue to add a check for hostname/path
shouldShowVulnerabilityPath(item) {
return Boolean(item.location.image || item.location.file || item.location.path);
},
createLocationString - add:
if (path) {
return path;
}
-
Update unit tests in https://gitlab.com/gitlab-org/gitlab/-/blob/master/ee/spec/frontend/security_dashboard/components/vulnerability_list_spec.js#L211 to check against a vulnerability with a path
Edited by Neil McCorrison