rails_surrogate_key_logging 1.4.0 → 1.5.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0c90ffc261a0cc4de8d88266fd00e0ce40c4c5a1802cf60b2fdaa72c6d345d61
-  data.tar.gz: 13dfb52ddf21fd49d28930043a4c8a3160c466fb2a03dfc8e9e369ad9b0007ab
+  metadata.gz: b9ce65ad0880f73135e03ce8be79ba2c3dc31b042d7cac0bf4c34acf15dcf96b
+  data.tar.gz: da6cdd954ca7dfa890f862dac29789fd78413f8106c629267990577713c2105a
 SHA512:
-  metadata.gz: cc2df820cec2c3c042552fd8e9e7f16c87b5c8592228a9f8c03e9c91e40417d5c0b7cf8cb2e15d729e56837e7ec6cccbb63a75e10c8cfd633ff55c6e029a7668
-  data.tar.gz: 80ce2c893ef9394acec270591f2c63d4f5603d701fd12eb8562a9b23a9d07555f8af0481a0e1df92692900e080bf826025cd20f77295514e8ac5fe0186a659cd
+  metadata.gz: 5127d29dc7d316edbcb8d957ddb2440eaea33012d5bbf8f04f56b93f8b93e4d85bb9c601a25915f771ddf8faf060e1ca332d1ae17fc7a9b8067c38a9df005da1
+  data.tar.gz: 7254fc40a0900861e7b35d44e549484acf42a0f913abf4a7b585c2522b7d28f116c05e21f276fa7f7f4c3ed20b0e33ba5535dd0235f768fa83118afd19f053db

data/lib/surrogate_key_logging/action_controller/params.rb

@@ -5,11 +5,11 @@ require 'active_support/concern'
 module SurrogateKeyLogging
   module ActionController
     module Params
-      extend ActiveSupport::Concern
+      extend ::ActiveSupport::Concern
 
       class_methods do
         def surrogate_params(*params, action: '*')
-          @surrogate_params ||= ActiveSupport::HashWithIndifferentAccess.new {|h,k| h[k] = [] }
+          @surrogate_params ||= ::ActiveSupport::HashWithIndifferentAccess.new {|h,k| h[k] = [] }
           params.each do |param|
             param = param.to_s
             @surrogate_params[action] << param

data/lib/surrogate_key_logging/action_controller.rb

@@ -2,7 +2,7 @@
 
 module SurrogateKeyLogging
   module ActionController
-    extend ActiveSupport::Autoload
+    extend ::ActiveSupport::Autoload
 
     autoload :LogSubscriber
 

data/lib/surrogate_key_logging/action_dispatch.rb

@@ -2,7 +2,7 @@
 
 module SurrogateKeyLogging
   module ActionDispatch
-    extend ActiveSupport::Autoload
+    extend ::ActiveSupport::Autoload
 
     autoload :ParamsFilter
     autoload :QueryStringFilter

data/lib/surrogate_key_logging/active_record/attributes.rb

@@ -5,7 +5,7 @@ require 'active_support/concern'
 module SurrogateKeyLogging
   module ActiveRecord
     module Attributes
-      extend ActiveSupport::Concern
+      extend ::ActiveSupport::Concern
 
       class_methods do
         def surrogate_parent_names(*names)

data/lib/surrogate_key_logging/active_record.rb

@@ -2,7 +2,7 @@
 
 module SurrogateKeyLogging
   module ActiveRecord
-    extend ActiveSupport::Autoload
+    extend ::ActiveSupport::Autoload
 
     autoload :Attributes
     autoload :LogSubscriber

data/lib/surrogate_key_logging/active_support.rb

@@ -1,36 +1,31 @@
 # frozen_string_literal: true
 
-require 'active_support/parameter_filter'
-
-# Add ability for @mask to be a class/instance/lambda/proc
-module ActiveSupport
-  class ParameterFilter
-    class CompiledFilter
-
-      def value_for_key(key, value, parents = [], original_params = nil) # rubocop:disable Metrics/AbcSize, Metrics/CyclomaticComplexity, Metrics/MethodLength, Metrics/PerceivedComplexity
-        parents.push(key) if deep_regexps
-        if regexps.any? { |r| r.match?(key.to_s) }
-          value = @mask.respond_to?(:call) ? @mask.call(key, value, parents, original_params) : @mask
-        elsif deep_regexps && (joined = parents.join('.')) && deep_regexps.any? { |r| r.match?(joined) } # rubocop:disable Lint/DuplicateBranch
-          value = @mask.respond_to?(:call) ? @mask.call(key, value, parents, original_params) : @mask
-        elsif value.is_a?(Hash)
-          value = call(value, parents, original_params)
-        elsif value.is_a?(Array)
-          # If we don't pop the current parent it will be duplicated as we
-          # process each array value.
-          parents.pop if deep_regexps
-          value = value.map { |v| value_for_key(key, v, parents, original_params) }
-          # Restore the parent stack after processing the array.
-          parents.push(key) if deep_regexps
-        elsif blocks.any?
-          key = key.dup if key.duplicable?
-          value = value.dup if value.duplicable?
-          blocks.each { |b| b.arity == 2 ? b.call(key, value) : b.call(key, value, original_params) }
-        end
+module SurrogateKeyLogging
+  module ActiveSupport
+    # Add ability for @mask to be a class/instance/lambda/proc
+    def value_for_key(key, value, parents = [], original_params = nil) # rubocop:disable Metrics/AbcSize, Metrics/CyclomaticComplexity, Metrics/MethodLength, Metrics/PerceivedComplexity
+      parents.push(key) if deep_regexps
+      if regexps.any? { |r| r.match?(key.to_s) }
+        value = @mask.respond_to?(:call) ? @mask.call(key, value, parents, original_params) : @mask
+      elsif deep_regexps && (joined = parents.join('.')) && deep_regexps.any? { |r| r.match?(joined) } # rubocop:disable Lint/DuplicateBranch
+        value = @mask.respond_to?(:call) ? @mask.call(key, value, parents, original_params) : @mask
+      elsif value.is_a?(Hash)
+        value = call(value, parents, original_params)
+      elsif value.is_a?(Array)
+        # If we don't pop the current parent it will be duplicated as we
+        # process each array value.
         parents.pop if deep_regexps
-        value
+        value = value.map { |v| value_for_key(key, v, parents, original_params) }
+        # Restore the parent stack after processing the array.
+        parents.push(key) if deep_regexps
+      elsif blocks.any?
+        key = key.dup if key.duplicable?
+        value = value.dup if value.duplicable?
+        blocks.each { |b| b.arity == 2 ? b.call(key, value) : b.call(key, value, original_params) }
       end
-
+      parents.pop if deep_regexps
+      value
     end
+    
   end
 end

data/lib/surrogate_key_logging/configuration.rb

@@ -1,11 +1,11 @@
 # frozen_string_literal: true
 
 module SurrogateKeyLogging
-  class Config < ActiveSupport::OrderedOptions
+  class Config < ::ActiveSupport::OrderedOptions
   end
 
   module Configuration
-    extend ActiveSupport::Concern
+    extend ::ActiveSupport::Concern
 
     def surrogate_key_logging
       SurrogateKeyLogging.config

data/lib/surrogate_key_logging/key_manager.rb

@@ -33,6 +33,7 @@ module SurrogateKeyLogging
 
     def call(_key, value, _parents = [], _original_params = nil)
       return "" if value.blank?
+      return value unless SurrogateKeyLogging.config.enabled
       surrogate = get(value)
       Rails.logger.tagged('SurrogateKeyLogging') { Rails.logger.info "Surrogate: `#{surrogate}`, value: `#{value}`" } if SurrogateKeyLogging.config.debug
       surrogate

data/lib/surrogate_key_logging/key_store.rb

@@ -2,7 +2,7 @@
 
 module SurrogateKeyLogging
   module KeyStore
-    extend ActiveSupport::Autoload
+    extend ::ActiveSupport::Autoload
 
     eager_autoload do
       autoload :Base

data/lib/surrogate_key_logging/sidekiq.rb

@@ -43,7 +43,7 @@ end
 module SurrogateKeyLogging
   class SidekiqMiddleware
 
-    def call(env)
+    def call(*_)
       yield
       SurrogateKeyLogging.reset
     end

data/lib/surrogate_key_logging/version.rb

@@ -4,8 +4,8 @@ module SurrogateKeyLogging
 
   module Version
     MAJOR = 1
-    MINOR = 4
-    PATCH = 0
+    MINOR = 5
+    PATCH = 1
 
   end
 

data/lib/surrogate_key_logging.rb

@@ -4,7 +4,7 @@ require 'active_support'
 
 # Container Module
 module SurrogateKeyLogging
-  extend ActiveSupport::Autoload
+  extend ::ActiveSupport::Autoload
 
   autoload :ActionController
   autoload :ActionDispatch
@@ -55,7 +55,7 @@ module SurrogateKeyLogging
     end
 
     def filter_for_attributes(attrs)
-      config.enabled ? ::ActiveSupport::ParameterFilter.new(attrs, mask: key_manager) : attrs
+      ::ActiveSupport::ParameterFilter.new(config.enabled ? attrs : [], mask: key_manager)
     end
 
     def key_store
@@ -63,7 +63,7 @@ module SurrogateKeyLogging
     end
 
     def filter_parameters(params)
-      config.enabled ? parameter_filter.filter(params) : params
+      parameter_filter.filter(params)
     end
 
     def surrogate_for(value)
@@ -84,6 +84,8 @@ module SurrogateKeyLogging
 
   end
 
+  require 'active_support/parameter_filter'
+  ::ActiveSupport::ParameterFilter::CompiledFilter.send(:prepend, ActiveSupport)
   KeyStore.eager_load!
   ::Rails::Application::Configuration.send(:include, Configuration)
   require 'surrogate_key_logging/engine'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rails_surrogate_key_logging
 version: !ruby/object:Gem::Version
-  version: 1.4.0
+  version: 1.5.1
 platform: ruby
 authors:
 - Taylor Yelverton
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-07-05 00:00:00.000000000 Z
+date: 2023-07-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: actionpack