Fix nil input in causing SetGroupSecretPushProtectionService to fail

Gal Katzrequested to merge
gkatz_fix_excluded_projects_ids_initializer into master

What does this MR do and why?

Adds an || operation between [] and the input value of excluded_projects_ids in SetSecretPushProtectionBaseService to ensure nil value in excluded_projects_ids will be translated to an empty list of projects to exclude.

This should resolve Bug: Bad input to SetGroupSecretPushProtectionS... (#514168 - closed) • Gal Katz • 17.9 and ensure that the worker can successfully handle the nil value which will be passed by Sidekiq’s sidekiq automatic job retry.

Relevant issue:

Bug: Bad input to SetGroupSecretPushProtectionS... (#514168 - closed) • Gal Katz • 17.9
See worker logs for additional details.

Merge request reports