Skip to content

Run override UUID logic for the added and fixed findings

Mehmet Emin INAC requested to merge 394703_run_override_uuids_logic into master

What does this MR do and why?

We need to run override UUIDs logic for the added & fixed findings for the MR security widget as their counterparts security_findings are saved into the database after running this logic. If we don't override the UUIDs for the added & fixed findings, we can not associate them with the security_findings for the following use case;

usecase-sequence-mix__1_

Note: We already have #390185 to use security_findings instead of report findings for the MR security report comparison logic which will remove all this complexity.

Verifying on production

The UUIDs of the added & fixed findings must be the same for https://gitlab.com/gitlab-org/govern/demos/sandbox/minac/static-security-reports/-/merge_requests/3/security_reports?type=sast.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Mehmet Emin INAC

Merge request reports