Support forwards compatibility for event sinks.

We can add new arguments and events without breaking compat as long as
the new arguments are added at the end. The new args will be silently
trimmed for sending to older event sinks, and new event types will be
ignored.

1 files changed, 11 insertions, 1 deletions

diff --git a/lib/chef/event_dispatch/dispatcher.rb b/lib/chef/event_dispatch/dispatcher.rb
index 370f8c51b4..9e17d78507 100644
--- a/lib/chef/event_dispatch/dispatcher.rb
+++ b/lib/chef/event_dispatch/dispatcher.rb
@@ -28,7 +28,17 @@ class Chef
       # Define a method that will be forwarded to all
       def self.def_forwarding_method(method_name)
         define_method(method_name) do |*args|
-          @subscribers.each { |s| s.send(method_name, *args) }
+          @subscribers.each do |s|
+            # Skip new/unsupported event names.
+            if s.respond_to?(method_name)
+              mth = s.method(method_name)
+              # Anything with a *args is arity -1, so use all arguments.
+              arity = mth.arity < 0 ? args.length : mth.arity
+              # Trim arguments to match what the subscriber expects to allow
+              # adding new arguments without breaking compat.
+              mth.call(*args.take(arity))
+            end
+          end
         end
       end