Skip to content

Fix Sidekiq warning for Integration and WebHook

Bojan Marjanovic requested to merge bmarjanovic/sidekiq-strict-args-fix into master

What does this MR do and why?

In this MR we are fixing the Sidekiq warning that the Integrations::ExecuteWorker, WebHookWorker and WebHooks::LogExecutionWorker causes in the logs:

Job arguments to WebHookWorker do not serialize to JSON safely. This will raise an error in
Sidekiq 7.0. See https://github.com/mperham/sidekiq/wiki/Best-Practices or raise an error today
by calling `Sidekiq.strict_args!` during Sidekiq initialization.
Job arguments to Integrations::ExecuteWorker do not serialize to JSON safely. This will raise an error in
Sidekiq 7.0. See https://github.com/mperham/sidekiq/wiki/Best-Practices or raise an error today
by calling `Sidekiq.strict_args!` during Sidekiq initialization.
Job arguments to WebHooks::LogExecutionWorker do not serialize to JSON safely. This will raise an error in
Sidekiq 7.0. See https://github.com/mperham/sidekiq/wiki/Best-Practices or raise an error today
by calling `Sidekiq.strict_args!` during Sidekiq initialization.

Screenshots or screen recordings

N/A

How to set up and validate locally

N/A

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