Skip to content

Fix default Ci config path

Marcos Rocha requested to merge mc_rocha-fix-default-ci-config-path-355010 into master

What does this MR do and why?

When the ci config file is deleted using the interface the ci_config_path attribute of the project is set to an empty string.

This MR updates the ci_config_path check to handle the empty string filename.

This Merge Request solves issue #355010 (closed)

Screenshots or screen recordings

These are strongly recommended to assist reviewers and reduce the time to merge your change.

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

  1. Set the ci config file to a non-default file and save the changes.
  2. Go to Security & Compliance -> Configuration
  3. SAST -> Enable SAST
  4. Create Merge Request
  5. Verify it's working correctly.
  6. Delete the ci config file name and save the changes.
  7. Repeat steps 2 to 5

MR acceptance checklist

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

Merge request reports

Loading