Merge pull request rails#47322 from jonathanhefner/secure_compare_rotator-decouple-messages-rotator

Detach `Messages::Rotator` from `SecureCompareRotator`

Author: jonathanhefner

activesupport/lib/active_support/secure_compare_rotator.rb

@@ -29,23 +29,28 @@ module ActiveSupport
   #   end
   class SecureCompareRotator
     include SecurityUtils
-    prepend Messages::Rotator
 
     InvalidMatch = Class.new(StandardError)
 
-    def initialize(value, **_options)
+    def initialize(value, on_rotation: nil)
       @value = value
+      @rotate_values = []
+      @on_rotation = on_rotation
     end
 
-    def secure_compare!(other_value, on_rotation: @on_rotation)
-      secure_compare(@value, other_value) ||
-        run_rotations(on_rotation) { |wrapper| wrapper.secure_compare!(other_value) } ||
-        raise(InvalidMatch)
+    def rotate(previous_value)
+      @rotate_values << previous_value
     end
 
-    private
-      def build_rotation(previous_value, **_options)
-        self.class.new(previous_value)
+    def secure_compare!(other_value, on_rotation: @on_rotation)
+      if secure_compare(@value, other_value)
+        true
+      elsif @rotate_values.any? { |value| secure_compare(value, other_value) }
+        on_rotation&.call
+        true
+      else
+        raise InvalidMatch
       end
+    end
   end
 end