diff options
author | Marek Pieczyk <codebaker95@gmail.com> | 2015-09-06 23:39:42 +0200 |
---|---|---|
committer | Marek Pieczyk <codebaker95@gmail.com> | 2015-09-08 23:28:12 +0200 |
commit | 26f37f7c401d95acb582dbb1d820e2333f84f217 (patch) | |
tree | 87e4545da06d77b6203e3daaa04c4dcd023e8370 /activejob/lib | |
parent | 997acb9aadab4df57b5a4ce88ddce3ec9d5f5f45 (diff) | |
download | rails-26f37f7c401d95acb582dbb1d820e2333f84f217.tar.gz rails-26f37f7c401d95acb582dbb1d820e2333f84f217.tar.bz2 rails-26f37f7c401d95acb582dbb1d820e2333f84f217.zip |
Properly log nested parameters to Active Job
Refactor arguments logging method for Active Job
Diffstat (limited to 'activejob/lib')
-rw-r--r-- | activejob/lib/active_job/logging.rb | 16 |
1 files changed, 15 insertions, 1 deletions
diff --git a/activejob/lib/active_job/logging.rb b/activejob/lib/active_job/logging.rb index 54774db601..d72e1bdfce 100644 --- a/activejob/lib/active_job/logging.rb +++ b/activejob/lib/active_job/logging.rb @@ -1,3 +1,4 @@ +require 'active_support/core_ext/hash/transform_values' require 'active_support/core_ext/string/filters' require 'active_support/tagged_logging' require 'active_support/logger' @@ -87,12 +88,25 @@ module ActiveJob def args_info(job) if job.arguments.any? ' with arguments: ' + - job.arguments.map { |arg| arg.try(:to_global_id).try(:to_s) || arg.inspect }.join(', ') + job.arguments.map { |arg| format(arg).inspect }.join(', ') else '' end end + def format(arg) + case arg + when Hash + arg.transform_values { |value| format(value) } + when Array + arg.map { |value| format(value) } + when GlobalID::Identification + arg.to_global_id rescue arg + else + arg + end + end + def scheduled_at(event) Time.at(event.payload[:job].scheduled_at).utc end |