about summary refs log tree commit diff
path: root/config
diff options
context:
space:
mode:
authorClaire <claire.github-309c@sitedethib.com>2022-01-28 00:43:56 +0100
committerGitHub <noreply@github.com>2022-01-28 00:43:56 +0100
commit03d59340da3ffc380d7b27169bdc887140d88bee (patch)
tree32f4a9835d860de7b3b2d0a42b42202c57f6d2e7 /config
parent14c69a535bcf5c96459a802093463e0bd1a8ef66 (diff)
Fix Sidekiq warnings about JSON serialization (#17381)
* Fix Sidekiq warnings about JSON serialization

This occurs on every symbol argument we pass, and every symbol key in hashes,
because Sidekiq expects strings instead.

See https://github.com/mperham/sidekiq/pull/5071

We do not need to change how workers parse their arguments because this has
not changed and we were already converting to symbols adequately or using
`with_indifferent_access`.

* Set Sidekiq to raise on unsafe arguments in test mode

In order to more easily catch issues that would produce warnings in production
code.
Diffstat (limited to 'config')
-rw-r--r--config/environments/test.rb3
1 files changed, 3 insertions, 0 deletions
diff --git a/config/environments/test.rb b/config/environments/test.rb
index a35cadcfa..ef3cb2e48 100644
--- a/config/environments/test.rb
+++ b/config/environments/test.rb
@@ -70,3 +70,6 @@ if ENV['PAM_ENABLED'] == 'true'
       env: { email: 'pam@example.com' }
     }
 end
+
+# Catch serialization warnings early
+Sidekiq.strict_args!