Skip to content

Draft: Remove misleading deprecation notice for the cve property

Olivier Gonzalez requested to merge gonzoyumo-master-patch-54421 into master

What does this MR do?

The cve property is not yet Deprecated totally.

The cve was initially used for 2 purposes: matching finding with "Feedback" and matching finding with "Remediations". The later usage has been replaced with vulnerabilities[].id as part of gitlab-org/gitlab#36777 (closed). The former is still pending and tracked with gitlab-org&2791.

While the schema still declares that property as required, the deprecation notice in the description misleads some users. It should be added back once the field is no longer required by the schema.

Availability and Testing

  • Review and add/update tests for this feature/bug

Approvals

Edited by Olivier Gonzalez

Merge request reports

Loading