rails_surrogate_key_logging 1.4.0 → 1.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0c90ffc261a0cc4de8d88266fd00e0ce40c4c5a1802cf60b2fdaa72c6d345d61
-  data.tar.gz: 13dfb52ddf21fd49d28930043a4c8a3160c466fb2a03dfc8e9e369ad9b0007ab
+  metadata.gz: f5d3a0747a79affaf71480538b5ca2567c962ddf03c6af176661ea082b7427e5
+  data.tar.gz: aec5af8fb44ca382501cef61b9d142328a8befa571df08821e4b9f5ef365490c
 SHA512:
-  metadata.gz: cc2df820cec2c3c042552fd8e9e7f16c87b5c8592228a9f8c03e9c91e40417d5c0b7cf8cb2e15d729e56837e7ec6cccbb63a75e10c8cfd633ff55c6e029a7668
-  data.tar.gz: 80ce2c893ef9394acec270591f2c63d4f5603d701fd12eb8562a9b23a9d07555f8af0481a0e1df92692900e080bf826025cd20f77295514e8ac5fe0186a659cd
+  metadata.gz: f8ec40ce47168317e745ed14d707ca7c8a35ce7bd9eec78c7224baf4bcaa5f2a92b8eeade6024a745aa66d018ea11bbb65fb5989461923d635df60cf0eb0f060
+  data.tar.gz: b245a066710ba8440584f4fb51588bf0a968e969ef41eef0ae52e739bfdf266cc40d63770ccc4866917a806437b7d1ebb8eb5bcfdc48f914ee47df44ca60f26e

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/version.rb

@@ -4,7 +4,7 @@ module SurrogateKeyLogging
 
   module Version
     MAJOR = 1
-    MINOR = 4
+    MINOR = 5
     PATCH = 0
 
   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,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rails_surrogate_key_logging
 version: !ruby/object:Gem::Version
-  version: 1.4.0
+  version: 1.5.0
 platform: ruby
 authors:
 - Taylor Yelverton