Fix security policy sync workers to use string keys for params
What does this MR do and why?
This MR fixes a regression introduced by Update SyncProjectPolicyWorker to handle events (!191788 - merged) where sidekiq params are accessed as symbol keys instead of string in Security::SyncProjectPolicyWorker. This MR also updates other workers where force_resync param in accessed as symbol instead of string.
References
Screenshots or screen recordings
| Before | After |
|---|---|
How to set up and validate locally
- Create a compliance framework
- Create a Merge Request approval policy scoped to that compliance framework:
policy_scope:
compliance_frameworks:
- id: 123
- Link a project to this compliance framework.
- Confirm that the policy appears in the Compliance Policies tab and is enforced on MRs in the linked project.
- Create an MR in the project and confirm that the approval is required
- Now navigate to:
Secure → Compliance Center → Project Tab - Edit the project and deselect/unlink the compliance framework.
- Go to the MR and confirm that the approval is not required
MR acceptance checklist
Evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.
Edited by Sashi Kumar Kumaresan