Update to use name field for training url graphql query
Changes | MR |
---|---|
BE | !84313 (merged) |
FE | this MR |
What does this MR do and why?
In this MR, we are updating the frontend to pass the name
field instead of the externalId
Background Info
We initially relied on the externalId
to make the API call to our provider. However we noticed this field data doesn't always provide the information we require, please see > #357008
Here is an instance, where the
externalId
is not what we need. Example > https://gitlab.com/stefania-demo/nodejs/-/security/vulnerabilities/39727260
So the backend MR > !84313 (merged) has updated their logic to handle and parse the more reliable name
field and use it to make the vendor API request
Screenshots or screen recordings
This MR does not introduce any UI/UX changes.
How to set up and validate locally
- Please make sure you have enabled the security training feature, docs; please enable both vendors.
- Navigate to the vulnerability page to view the training link, docs
- Note, we currently only support
cwe
identifiers. - We are making the graphql query with the identifier
name
, NOTexternalId
The identifier name
|
Passing the identifier name (NOT externalId ) |
---|---|
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 #357727 (closed)