Update graphql type to match expected response structure
Overview
As part of the work to display SAST and Code Quality findings in merge request diff view (see parent epic), we have to update the FindingType
graphql type to match expected response structure.
The current design (see particularly this overview mockup) requires a few more fields to be present in the structure of the FindingType
objects, namely:
-
location
object (to be able to present the file path and starting line). -
identifiers
object (so it can present more information on this type of vulnerability).
Proposal
To ensure both location
and identifiers
objects are available in the response of getMRSecurityReport
query, they have to be included in FindingType
graphql type. If no corresponding types and resolvers exist for those kind of objects already, they also need to be created to ensure appropriate representation of them.
Edited by Ahmed Hemdan