Skip to content

fix(security): handle security reports gracefully

Dheeraj Joshi requested to merge djadmin-fix-security-report-regression into main

Issue

#1255 (closed)

Description

Fixes a regression with security findings. It looks like the mergeReports util was changed recently for better error handling, and got a regression.

Related Issues

Related thread: https://gitlab.slack.com/archives/C8S0HHM44/p1707931787702659

How has this been tested?

Screenshots (if appropriate)

Checkout this MR: gitlab!144879 (closed)

Before After
image image

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation
  • Chore (Related to CI or Packaging to platforms)
  • Test gap
Edited by Fernando Cardenas

Merge request reports