activejob 6.0.3.2 → 6.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5aa67f26abfc254d6855b0067f396a253241fdc74df407f4261d3dacb7235c48
-  data.tar.gz: d42e8410e3aaabd94cc68fb3ca300fd30a5a0ed2806be1ffcb036f1accd62c9a
+  metadata.gz: 6e8ecd60ff8f673ab2711ef5b6b634ce5828160113e8ac5eec732d71bc691660
+  data.tar.gz: 0207aac145fea7328f0afbe4680413712bbf1f1bd7a400d0b19f178ecf21024f
 SHA512:
-  metadata.gz: 2d9937f4a3f5c0a3b062f2f6ad4db4db30ce9893c4423f2e49c7fedcb8ecd01fa101e1ee36dcbe670d2538bc440f8bd83dc4bfb3114515bd84d3d68202211ae2
-  data.tar.gz: 02baf8feb8d09b07c190ff90c40d40d880ad6eed7ecd672fff080910839ea5139a4f5deaeb2006b0f5269cda5d587e5cf9b7890e368e4d43f0e30b7c267516c7
+  metadata.gz: bf88c0a034e81a6d1a75fa95b37b790647eac95a4f3bfdef7b93bb0880d80e7e040a2eb9afe735544ed11ffdd7337eb7f5bcdf8fe2f3c38045ed5319b8bf5e02
+  data.tar.gz: 4fbc6a306f542ba1dbb749e1defb085bd52c691b6287c6ef3e10a8bfd527f8e85914c162b5728f47e08271f6367f182195cf30fb6a7c8ba4c2b5af211b4e2b7d

data/CHANGELOG.md

@@ -1,203 +1,130 @@
-## Rails 6.0.3.2 (June 17, 2020) ##
+## Rails 6.1.0 (December 09, 2020) ##
 
-*   No changes.
+*   Recover nano precision when serializing `Time`, `TimeWithZone` and `DateTime` objects.
 
+    *Alan Tan*
 
-## Rails 6.0.3.1 (May 18, 2020) ##
+*   Deprecate `config.active_job.return_false_on_aborted_enqueue`.
 
-*   No changes.
+    *Rafael Mendonça França*
 
+*   Return `false` when enqueuing a job is aborted.
 
-## Rails 6.0.3 (May 06, 2020) ##
+    *Rafael Mendonça França*
 
 *   While using `perform_enqueued_jobs` test helper enqueued jobs must be stored for the later check with
     `assert_enqueued_with`.
 
     *Dmitry Polushkin*
 
-*   Add queue name support to Que adapter
+*   `ActiveJob::TestCase#perform_enqueued_jobs` without a block removes performed jobs from the queue.
 
-    *Brad Nauta*, *Wojciech Wnętrzak*
-
-
-## Rails 6.0.2.2 (March 19, 2020) ##
-
-*   No changes.
-
-
-## Rails 6.0.2.1 (December 18, 2019) ##
-
-*   No changes.
-
-
-## Rails 6.0.2 (December 13, 2019) ##
-
-*   Allow Sidekiq access to the underlying job class.
-
-    By having access to the Active Job class, Sidekiq can get access to any `sidekiq_options` which
-    have been set on that Active Job type and serialize those options into Redis.
-
-    https://github.com/mperham/sidekiq/blob/master/Changes.md#60
-
-    *Mike Perham*
-
-
-## Rails 6.0.1 (November 5, 2019) ##
-
-*   No changes.
-
-
-## Rails 6.0.0 (August 16, 2019) ##
-
-*   `assert_enqueued_with` and `assert_performed_with` can now test jobs with relative delay.
-
-    *Vlado Cingel*
-
-
-## Rails 6.0.0.rc2 (July 22, 2019) ##
-
-*   No changes.
-
-
-## Rails 6.0.0.rc1 (April 24, 2019) ##
-
-*   Use individual execution counters when calculating retry delay.
-
-    *Patrik Bóna*
-
-*   Make job argument assertions with `Time`, `ActiveSupport::TimeWithZone`, and `DateTime` work by dropping microseconds. Microsecond precision is lost during serialization.
-
-    *Gannon McGibbon*
+    That way the helper can be called multiple times and not perform a job invocation multiple times.
 
+    ```ruby
+    def test_jobs
+      HelloJob.perform_later("rafael")
+      perform_enqueued_jobs # only runs with "rafael"
+      HelloJob.perform_later("david")
+      perform_enqueued_jobs # only runs with "david"
+    end
+    ```
 
-## Rails 6.0.0.beta3 (March 11, 2019) ##
-
-*   No changes.
-
-
-## Rails 6.0.0.beta2 (February 25, 2019) ##
-
-*   No changes.
+    *Étienne Barrié*
 
+*   `ActiveJob::TestCase#perform_enqueued_jobs` will no longer perform retries:
 
-## Rails 6.0.0.beta1 (January 18, 2019) ##
+    When calling `perform_enqueued_jobs` without a block, the adapter will
+    now perform jobs that are **already** in the queue. Jobs that will end up in
+    the queue afterwards won't be performed.
 
-*   Return false instead of the job instance when `enqueue` is aborted.
+    This change only affects `perform_enqueued_jobs` when no block is given.
 
-    This will be the behavior in Rails 6.1 but it can be controlled now with
-    `config.active_job.return_false_on_aborted_enqueue`.
+    *Edouard Chin*
 
-    *Kir Shatrov*
+*   Add queue name support to Que adapter.
 
-*   Keep executions for each specific declaration
+    *Brad Nauta*, *Wojciech Wnętrzak*
 
-    Each `retry_on` declaration has now its own specific executions counter. Before it was
-    shared between all executions of a job.
+*   Don't run `after_enqueue` and `after_perform` callbacks if the callback chain is halted.
 
-    *Alberto Almagro*
+        class MyJob < ApplicationJob
+          before_enqueue { throw(:abort) }
+          after_enqueue { # won't enter here anymore }
+        end
 
-*   Allow all assertion helpers that have a `only` and `except` keyword to accept
-    Procs.
+    `after_enqueue` and `after_perform` callbacks will no longer run if the callback chain is halted.
+    This behaviour is a breaking change and won't take effect until Rails 6.2.
+    To enable this behaviour in your app right now, you can add in your app's configuration file
+    `config.active_job.skip_after_callbacks_if_terminated = true`.
 
     *Edouard Chin*
 
-*   Restore `HashWithIndifferentAccess` support to `ActiveJob::Arguments.deserialize`.
+*   Fix enqueuing and performing incorrect logging message.
 
-    *Gannon McGibbon*
+    Jobs will no longer always log "Enqueued MyJob" or "Performed MyJob" when they actually didn't get enqueued/performed.
 
-*   Include deserialized arguments in job instances returned from
-    `assert_enqueued_with` and `assert_performed_with`
+    ```ruby
+      class MyJob < ApplicationJob
+        before_enqueue { throw(:abort) }
+      end
 
-    *Alan Wu*
+      MyJob.perform_later # Will no longer log "Enqueued MyJob" since job wasn't even enqueued through adapter.
+    ```
 
-*   Allow `assert_enqueued_with`/`assert_performed_with` methods to accept
-    a proc for the `args` argument. This is useful to check if only a subset of arguments
-    matches your expectations.
+    A new message will be logged in case a job couldn't be enqueued, either because the callback chain was halted or
+    because an exception happened during enqueuing. (i.e. Redis is down when you try to enqueue your job)
 
     *Edouard Chin*
 
-*   `ActionDispatch::IntegrationTest` includes `ActiveJob::TestHelper` module by default.
-
-    *Ricardo Díaz*
-
-*   Added `enqueue_retry.active_job`, `retry_stopped.active_job`, and `discard.active_job` hooks.
-
-    *steves*
+*   Add an option to disable logging of the job arguments when enqueuing and executing the job.
 
-*   Allow `assert_performed_with` to be called without a block.
+        class SensitiveJob < ApplicationJob
+          self.log_arguments = false
 
-    *bogdanvlviv*
+          def perform(my_sensitive_argument)
+          end
+        end
 
-*   Execution of `assert_performed_jobs`, and `assert_no_performed_jobs`
-    without a block should respect passed `:except`, `:only`, and `:queue` options.
+    When dealing with sensitive arguments as password and tokens it is now possible to configure the job
+    to not put the sensitive argument in the logs.
 
-    *bogdanvlviv*
+    *Rafael Mendonça França*
 
-*   Allow `:queue` option to job assertions and helpers.
+*   Changes in `queue_name_prefix` of a job no longer affects all other jobs.
 
-    *bogdanvlviv*
+    Fixes #37084.
 
-*   Allow `perform_enqueued_jobs` to be called without a block.
+    *Lucas Mansur*
 
-    Performs all of the jobs that have been enqueued up to this point in the test.
+*   Allow `Class` and `Module` instances to be serialized.
 
     *Kevin Deisz*
 
-*   Move `enqueue`/`enqueue_at` notifications to an around callback.
-
-    Improves timing accuracy over the old after callback by including
-    time spent writing to the adapter's IO implementation.
-
-    *Zach Kemp*
-
-*   Allow call `assert_enqueued_with` with no block.
+*   Log potential matches in `assert_enqueued_with` and `assert_performed_with`.
 
-    Example:
-    ```
-    def test_assert_enqueued_with
-      MyJob.perform_later(1,2,3)
-      assert_enqueued_with(job: MyJob, args: [1,2,3], queue: 'low')
-
-      MyJob.set(wait_until: Date.tomorrow.noon).perform_later
-      assert_enqueued_with(job: MyJob, at: Date.tomorrow.noon)
-    end
-    ```
-
-    *bogdanvlviv*
-
-*   Allow passing multiple exceptions to `retry_on`, and `discard_on`.
-
-    *George Claghorn*
-
-*   Pass the error instance as the second parameter of block executed by `discard_on`.
+    *Gareth du Plooy*
 
-    Fixes #32853.
+*   Add `at` argument to the `perform_enqueued_jobs` test helper.
 
-    *Yuji Yaginuma*
+    *John Crepezzi*, *Eileen Uchitelle*
 
-*   Remove support for Qu gem.
-
-    Reasons are that the Qu gem wasn't compatible since Rails 5.1,
-    gem development was stopped in 2014 and maintainers have
-    confirmed its demise. See issue #32273
-
-    *Alberto Almagro*
-
-*   Add support for timezones to Active Job.
-
-    Record what was the current timezone in effect when the job was
-    enqueued and then restore when the job is executed in same way
-    that the current locale is recorded and restored.
+*   `assert_enqueued_with` and `assert_performed_with` can now test jobs with relative delay.
 
-    *Andrew White*
+    *Vlado Cingel*
 
-*   Rails 6 requires Ruby 2.5.0 or newer.
+*   Add jitter to `ActiveJob::Exceptions.retry_on`.
 
-    *Jeremy Daer*, *Kasper Timm Hansen*
+    `ActiveJob::Exceptions.retry_on` now uses a random amount of jitter in order to
+    prevent the [thundering herd effect](https://en.wikipedia.org/wiki/Thundering_herd_problem). Defaults to
+    15% (represented as 0.15) but overridable via the `:jitter` option when using `retry_on`.
+    Jitter is applied when an `Integer`, `ActiveSupport::Duration` or `:exponentially_longer`, is passed to the `wait` argument in `retry_on`.
 
-*   Add support to define custom argument serializers.
+    ```ruby
+    retry_on(MyError, wait: :exponentially_longer, jitter: 0.30)
+    ```
 
-    *Evgenii Pecherkin*, *Rafael Mendonça França*
+    *Anthony Ross*
 
 
-Please check [5-2-stable](https://github.com/rails/rails/blob/5-2-stable/activejob/CHANGELOG.md) for previous changes.
+Please check [6-0-stable](https://github.com/rails/rails/blob/6-0-stable/activejob/CHANGELOG.md) for previous changes.

data/MIT-LICENSE

@@ -1,4 +1,4 @@
-Copyright (c) 2014-2019 David Heinemeier Hansson
+Copyright (c) 2014-2020 David Heinemeier Hansson
 
 Permission is hereby granted, free of charge, to any person obtaining
 a copy of this software and associated documentation files (the

data/README.md

@@ -95,9 +95,6 @@ their gem, or as a stand-alone gem. For discussion about this see the
 following PRs: [23311](https://github.com/rails/rails/issues/23311#issuecomment-176275718),
 [21406](https://github.com/rails/rails/pull/21406#issuecomment-138813484), and [#32285](https://github.com/rails/rails/pull/32285).
 
-## Auxiliary gems
-
-* [activejob-stats](https://github.com/seuros/activejob-stats)
 
 ## Download and installation
 
@@ -111,6 +108,7 @@ Source code can be downloaded as part of the Rails project on GitHub:
 
 * https://github.com/rails/rails/tree/master/activejob
 
+
 ## License
 
 Active Job is released under the MIT license:

data/lib/active_job.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 #--
-# Copyright (c) 2014-2019 David Heinemeier Hansson
+# Copyright (c) 2014-2020 David Heinemeier Hansson
 #
 # Permission is hereby granted, free of charge, to any person obtaining
 # a copy of this software and associated documentation files (the

data/lib/active_job/base.rb

@@ -8,7 +8,9 @@ require "active_job/enqueuing"
 require "active_job/execution"
 require "active_job/callbacks"
 require "active_job/exceptions"
+require "active_job/log_subscriber"
 require "active_job/logging"
+require "active_job/instrumentation"
 require "active_job/timezones"
 require "active_job/translation"
 
@@ -68,6 +70,7 @@ module ActiveJob #:nodoc:
     include Callbacks
     include Exceptions
     include Logging
+    include Instrumentation
     include Timezones
     include Translation
 

data/lib/active_job/callbacks.rb

@@ -1,6 +1,8 @@
 # frozen_string_literal: true
 
 require "active_support/callbacks"
+require "active_support/core_ext/object/with_options"
+require "active_support/core_ext/module/attribute_accessors"
 
 module ActiveJob
   # = Active Job Callbacks
@@ -27,16 +29,25 @@ module ActiveJob
     end
 
     included do
-      define_callbacks :perform
-      define_callbacks :enqueue
+      class_attribute :return_false_on_aborted_enqueue, instance_accessor: false, instance_predicate: false, default: false
+      singleton_class.deprecate :return_false_on_aborted_enqueue, :return_false_on_aborted_enqueue=
+      cattr_accessor :skip_after_callbacks_if_terminated, instance_accessor: false, default: false
 
-      class_attribute :return_false_on_aborted_enqueue, instance_accessor: false, instance_predicate: false
-      self.return_false_on_aborted_enqueue = false
+      with_options(skip_after_callbacks_if_terminated: skip_after_callbacks_if_terminated) do
+        define_callbacks :perform
+        define_callbacks :enqueue
+      end
     end
 
     # These methods will be included into any Active Job object, adding
     # callbacks for +perform+ and +enqueue+ methods.
     module ClassMethods
+      def inherited(klass)
+        klass.get_callbacks(:enqueue).config[:skip_after_callbacks_if_terminated] = skip_after_callbacks_if_terminated
+        klass.get_callbacks(:perform).config[:skip_after_callbacks_if_terminated] = skip_after_callbacks_if_terminated
+        super
+      end
+
       # Defines a callback that will get called right before the
       # job's perform method is executed.
       #
@@ -91,6 +102,19 @@ module ActiveJob
       #     end
       #   end
       #
+      # You can access the return value of the job only if the execution wasn't halted.
+      #
+      #   class VideoProcessJob < ActiveJob::Base
+      #     around_perform do |job, block|
+      #       value = block.call
+      #       puts value # => "Hello World!"
+      #     end
+      #
+      #     def perform
+      #       "Hello World!"
+      #     end
+      #   end
+      #
       def around_perform(*filters, &blk)
         set_callback(:perform, :around, *filters, &blk)
       end
@@ -154,5 +178,21 @@ module ActiveJob
         set_callback(:enqueue, :around, *filters, &blk)
       end
     end
+
+    private
+      def halted_callback_hook(_filter, name) # :nodoc:
+        return super unless %i(enqueue perform).include?(name.to_sym)
+        callbacks = public_send("_#{name}_callbacks")
+
+        if !self.class.skip_after_callbacks_if_terminated && callbacks.any? { |c| c.kind == :after }
+          ActiveSupport::Deprecation.warn(<<~EOM)
+            In Rails 6.2, `after_enqueue`/`after_perform` callbacks no longer run if `before_enqueue`/`before_perform` respectively halts with `throw :abort`.
+            To enable this behavior, uncomment the `config.active_job.skip_after_callbacks_if_terminated` config
+            in the new 6.1 framework defaults initializer.
+          EOM
+        end
+
+        super
+      end
   end
 end

data/lib/active_job/core.rb

@@ -85,6 +85,7 @@ module ActiveJob
       @priority   = self.class.priority
       @executions = 0
       @exception_executions = {}
+      @timezone   = Time.zone&.name
     end
     ruby2_keywords(:initialize) if respond_to?(:ruby2_keywords, true)
 
@@ -101,7 +102,7 @@ module ActiveJob
         "executions" => executions,
         "exception_executions" => exception_executions,
         "locale"     => I18n.locale.to_s,
-        "timezone"   => Time.zone.try(:name),
+        "timezone"   => timezone,
         "enqueued_at" => Time.now.utc.iso8601
       }
     end
@@ -141,7 +142,7 @@ module ActiveJob
       self.executions           = job_data["executions"]
       self.exception_executions = job_data["exception_executions"]
       self.locale               = job_data["locale"] || I18n.locale.to_s
-      self.timezone             = job_data["timezone"] || Time.zone.try(:name)
+      self.timezone             = job_data["timezone"] || Time.zone&.name
       self.enqueued_at          = job_data["enqueued_at"]
     end
 

data/lib/active_job/enqueuing.rb

@@ -54,9 +54,9 @@ module ActiveJob
 
       run_callbacks :enqueue do
         if scheduled_at
-          self.class.queue_adapter.enqueue_at self, scheduled_at
+          queue_adapter.enqueue_at self, scheduled_at
         else
-          self.class.queue_adapter.enqueue self
+          queue_adapter.enqueue self
         end
 
         successfully_enqueued = true
@@ -65,17 +65,7 @@ module ActiveJob
       if successfully_enqueued
         self
       else
-        if self.class.return_false_on_aborted_enqueue
-          false
-        else
-          ActiveSupport::Deprecation.warn(
-            "Rails 6.1 will return false when the enqueuing is aborted. Make sure your code doesn't depend on it" \
-            " returning the instance of the job and set `config.active_job.return_false_on_aborted_enqueue = true`" \
-            " to remove the deprecations."
-          )
-
-          self
-        end
+        false
       end
     end
   end