sqreen 1.7.1 → 1.7.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: b7c0d876ee5f24abd72e726834023d2f9629169f
-  data.tar.gz: 71bd73ae7b3513b8727286abba6488ccd704d341
+  metadata.gz: 5741aa7c3b34891c2a790746a3bdaa41b89af92b
+  data.tar.gz: bf1170f3d02dc780352e4c55a5f26862ce0a340e
 SHA512:
-  metadata.gz: 395f0a6db93a2ce37ab1500c3bf16aa5f44ab93cb94701a7bc1bbf7803d68ff2380b05dbde439c89c19ba6e80a3425b99c227c1304f1ea5220d57e15bb5fc959
-  data.tar.gz: ce8eaa36bc6224afe30e1838183562e11e724415901de1fd91c243af6b94367e4d044e41c73ffa0e81fb8a953a0598c74b32185b915d616a8d1dcb988d3ae7de
+  metadata.gz: 87ca7a7ae12ab3b762c12b4c051d0fbf7bce0f16a4b11d9b80462aff8ba01a2f3ce4c8ddcd41c41cc9c93ddb4f920384f6812410b0c4089ffd3c78e2812c4490
+  data.tar.gz: d93a22ee357be92f2fe75b877f2be5af7d7b947ca605102743a241c7592b300d8fd43ad2cb19c7cc707bb832512edf56a5df6fbaf05851dc5fb64b91b123bf8f

data/lib/sqreen/instrumentation.rb

@@ -61,6 +61,8 @@ module Sqreen
 
         returns = []
         callbacks.each do |cb|
+          # If record_request is part of callbacks we should filter after it ran
+          next if cb.whitelisted?
           rule = cb.rule_name if cb.respond_to?(:rule_name)
           Sqreen.log.debug { "running pre cb #{cb}" }
           Sqreen::PerformanceNotifications.instrument("Callbacks/#{rule || cb.class.name}/pre") do

data/lib/sqreen/rules_callbacks/binding_accessor_matcher.rb

@@ -57,6 +57,7 @@ module Sqreen
             next unless val.respond_to?(:each)
             next if val.respond_to?(:seek)
             val.each do |v|
+              next if !v.is_a?(String) || (!matcher.min_size.nil? && v.size < matcher.min_size)
               next if matcher.match(v).nil?
               infos = {
                 'id' => id,

data/lib/sqreen/rules_callbacks/matcher_rule.rb

@@ -7,6 +7,7 @@ module Sqreen
   module Rules
     # matcher behavior
     module Matcher
+      attr_reader :min_size
       def self.prepare_re_pattern(value, options, case_sensitive)
         res = 0
         res |= Regexp::MULTILINE  if options.include?('multiline')
@@ -33,6 +34,7 @@ module Sqreen
           'equals'.freeze    => lambda { |value, str| str == value           },
         }
 
+        sizes = []
         patterns.each do |entry|
           next unless entry
           type = entry['type']
@@ -56,25 +58,28 @@ module Sqreen
             end
             @string[opt] = { :ci => [], :cs => [] } unless @string.key?(opt)
             @string[opt][case_type] << val
-
+            sizes << entry.fetch('min_length') { val.size }
           when 'regexp'
             pattern = Matcher.prepare_re_pattern(val, opt, case_sensitive)
             next unless pattern
             @regexp_patterns << pattern
+            sizes << entry['min_length']
           else
             raise Sqreen::Exception, "No such matcher type #{type}"
           end
         end
 
+        @min_size = sizes.min unless sizes.any?(&:nil?)
+
         return unless [@regexp_patterns, @string].map(&:empty?).all?
         msg = "no key 'regexp' nor 'match' in data (had #{@data.keys})"
         raise Sqreen::Exception, msg
       end
 
       def match(str)
-        return if str.nil? || str.empty?
-        str = enforce_encoding(str)
-        istr = str.downcase
+        return if str.nil? || str.empty? || !str.is_a?(String)
+        str = enforce_encoding(str) unless str.ascii_only?
+        istr = str.downcase unless @string.empty?
 
         @string.each do |type, cases|
           fun = @funs[type]
@@ -100,7 +105,6 @@ module Sqreen
           end
         else
           @regexp_patterns.each do |p|
-            warn(Encoding.compatible?(p, str).inspect)
             return p if p.match(str)
           end
         end
@@ -110,8 +114,6 @@ module Sqreen
       private
 
       def enforce_encoding(str)
-        return str unless str.is_a?(String)
-        return str if str.ascii_only?
         encoded8bit = str.encoding.name == 'ASCII-8BIT'
         return str if !encoded8bit && str.valid_encoding?
         str.chars.map do |v|

data/lib/sqreen/version.rb

@@ -1,5 +1,5 @@
 # Copyright (c) 2015 Sqreen. All Rights Reserved.
 # Please refer to our terms for more information: https://www.sqreen.io/terms.html
 module Sqreen
-  VERSION = '1.7.1'.freeze
+  VERSION = '1.7.2'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sqreen
 version: !ruby/object:Gem::Version
-  version: 1.7.1
+  version: 1.7.2
 platform: ruby
 authors:
 - Sqreen
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-07-10 00:00:00.000000000 Z
+date: 2017-07-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: execjs