Place Vulnerabilities::FindingsPreloader behind feedback deprecation FF
What does this MR do and why?
Vulnerabilities::FindingsPreloader preloads vulnerability_feedbacks on serialised requests for performance improvements. This behaviour is not required with the deprecation of Vulnerability Feedbacks, so this MR checks if the deprecation FF is enabled, and prevents the preload accordingly.
Validation
- The following graphql query should function successfully whether the flag is enabled or not, as the flag does not currently disable the serialisation of vulnerability feedbacks.
{
group(fullPath: "flightjs") {
projects {
nodes {
pipelines {
nodes {
securityReportFindings {
nodes {
uuid
}
}
}
}
}
}
}
}
- Requests to
http://localhost:3000/api/v4/projects/6/vulnerability_findingsShould work with the flag enabled or not when supplied with a valid api token.
MR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.
-
I have evaluated the MR acceptance checklist for this MR.
Related to #384221 (closed)