statsig 1.4.0 → 1.6.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3ac164a57fadf37c1c9ce2b978df30da43901cc3b53e03e1b3e52a72e19981cb
-  data.tar.gz: aad0387cd6a80bfd3cbf501118466f444790ec387b45a96d562a3256331ae034
+  metadata.gz: 325d5430e48a3d71b68d2e284e20b66b0052bf1a4d9e99e7a92a84069b680ab8
+  data.tar.gz: b227e1fede44928035b1a03ebc8e8c006425466b50b007f89c452f6a47b51cd6
 SHA512:
-  metadata.gz: f7eeefd82a45217becadee9783d2616de5e6997d29236fecea526947a34e54f921125fbf018f57a1fbf2f7560c00d80721f632a9498e0f065b6914b035beef5f
-  data.tar.gz: 24b8c55535b7656109b669d898351e536953c83ecc6858e94eabacbdb36f8c9e600b55bb9cd06fc8b043a84305f351e645c07a673a85c1688668de6181c125f3
+  metadata.gz: 28017886b1a5d3f5c37e36680f0dfbd300cf5a0f6e0678430af69631486bc88689a622d74a63da1687ef7d610d612227d68cb58b85a7572b7cc5cfcaee0794dd
+  data.tar.gz: 10a66e7674df116a65351b97b8d27055bacd46f021309c3256bc5605fe3a972416e2c595135262fc9d18117e447a5038db3ebe1b89cb71495f308430fe4d2a32

data/lib/evaluation_helpers.rb

@@ -6,19 +6,11 @@ module EvaluationHelpers
     func.call(a.to_f, b.to_f) rescue false
   end
 
-  # returns true if array contains value, ignoring case when comparing strings
-  def self.array_contains(array, value, ignore_case)
-    return false unless array.is_a?(Array) && !value.nil?
-    if value.is_a?(String) && match_string_in_array(array, value, ignore_case, ->(a, b) { a == b })
-      return true
-    end
-    return array.include?(value)
-  end
-
   # returns true if array has any element that evaluates to true with value using func lambda, ignoring case
   def self.match_string_in_array(array, value, ignore_case, func)
-    return false unless array.is_a?(Array) && value.is_a?(String)
-    array.any?{ |s| s.is_a?(String) && ((ignore_case && func.call(value.downcase, s.downcase)) || func.call(value, s)) } rescue false
+    return false unless array.is_a?(Array) && !value.nil?
+    str_value = value.to_s
+    array.any?{ |s| !s.nil? && ((ignore_case && func.call(str_value.downcase, s.to_s.downcase)) || func.call(str_value, s.to_s)) } rescue false
   end
 
   def self.compare_times(a, b, func)

data/lib/evaluator.rb

@@ -84,10 +84,10 @@ class Evaluator
       return $fetch_from_server if other_gate_result == $fetch_from_server
       return type == 'pass_gate' ? other_gate_result.gate_value : !other_gate_result.gate_value
     when 'ip_based'
-      value = get_value_from_user(user, field) || get_value_from_ip(user&.value_lookup['ip'], field)
+      value = get_value_from_user(user, field) || get_value_from_ip(user, field)
       return $fetch_from_server if value == $fetch_from_server
     when 'ua_based'
-      value = get_value_from_user(user, field) || get_value_from_ua(user&.value_lookup['userAgent'], field)
+      value = get_value_from_user(user, field) || get_value_from_ua(user, field)
       return $fetch_from_server if value == $fetch_from_server
     when 'user_field'
       value = get_value_from_user(user, field)
@@ -138,13 +138,13 @@ class Evaluator
 
       # array operations
     when 'any'
-      return EvaluationHelpers::array_contains(target, value, true)
+      return EvaluationHelpers::match_string_in_array(target, value, true, ->(a, b) { a == b })
     when 'none'
-      return !EvaluationHelpers::array_contains(target, value, true)
+      return !EvaluationHelpers::match_string_in_array(target, value, true, ->(a, b) { a == b })
     when 'any_case_sensitive'
-      return EvaluationHelpers::array_contains(target, value, false)
+      return EvaluationHelpers::match_string_in_array(target, value, false, ->(a, b) { a == b })
     when 'none_case_sensitive'
-      return !EvaluationHelpers::array_contains(target, value, false)
+      return !EvaluationHelpers::match_string_in_array(target, value, false, ->(a, b) { a == b })
 
       #string
     when 'str_starts_with_any'
@@ -153,6 +153,8 @@ class Evaluator
       return EvaluationHelpers::match_string_in_array(target, value, true, ->(a, b) { a.end_with?(b) })
     when 'str_contains_any'
       return EvaluationHelpers::match_string_in_array(target, value, true, ->(a, b) { a.include?(b) })
+    when 'str_contains_none'
+      return !EvaluationHelpers::match_string_in_array(target, value, true, ->(a, b) { a.include?(b) })
     when 'str_matches'
       return (value.is_a?(String) && !(value =~ Regexp.new(target)).nil? rescue false)
     when 'eq'
@@ -177,13 +179,22 @@ class Evaluator
 
     user_lookup_table = user&.value_lookup
     return nil unless user_lookup_table.is_a?(Hash)
-    return user_lookup_table[field.downcase] if user_lookup_table.has_key?(field.downcase)
+    return user_lookup_table[field.downcase] if user_lookup_table.has_key?(field.downcase) && !user_lookup_table[field.downcase].nil?
 
     user_custom = user_lookup_table['custom']
-    return nil unless user_custom.is_a?(Hash)
-    user_custom.each do |key, value|
-      return value if key.downcase.casecmp?(field.downcase)
+    if user_custom.is_a?(Hash)
+      user_custom.each do |key, value|
+        return value if key.downcase.casecmp?(field.downcase) && !value.nil?
+      end
+    end
+
+    private_attributes = user_lookup_table['privateAttributes']
+    if private_attributes.is_a?(Hash)
+      private_attributes.each do |key, value|
+        return value if key.downcase.casecmp?(field.downcase) && !value.nil?
+      end
     end
+
     nil
   end
 
@@ -197,17 +208,19 @@ class Evaluator
     nil
   end
 
-  def get_value_from_ip(ip, field)
-    return nil unless ip.is_a?(String) && field.is_a?(String)
+  def get_value_from_ip(user, field)
+    return nil unless user.is_a?(StatsigUser) && field.is_a?(String) && field.downcase == 'country'
+    ip = get_value_from_user(user, 'ip')
+    return nil unless ip.is_a?(String)
 
-    if field.downcase != 'country'
-      return $fetch_from_server
-    end
     CountryLookup.lookup_ip_string(ip)
   end
 
-  def get_value_from_ua(ua, field)
-    return nil unless ua.is_a?(String) && field.is_a?(String)
+  def get_value_from_ua(user, field)
+    return nil unless user.is_a?(StatsigUser) && field.is_a?(String)
+    ua = get_value_from_user(user, 'userAgent')
+    return nil unless ua.is_a?(String)
+
     parsed = @ua_parser.parse ua
     os = parsed.os
     case field.downcase

data/lib/network.rb

@@ -39,7 +39,7 @@ class Network
 
   def check_gate(user, gate_name)
     begin
-      request_body = JSON.generate({'user' => user&.serialize, 'gateName' => gate_name})
+      request_body = JSON.generate({'user' => user&.serialize(false), 'gateName' => gate_name})
       response = post_helper('check_gate', request_body)
       return JSON.parse(response.body) unless response.nil?
       false
@@ -50,7 +50,7 @@ class Network
 
   def get_config(user, dynamic_config_name)
     begin
-      request_body = JSON.generate({'user' => user&.serialize, 'configName' => dynamic_config_name})
+      request_body = JSON.generate({'user' => user&.serialize(false), 'configName' => dynamic_config_name})
       response = post_helper('get_config', request_body)
       return JSON.parse(response.body) unless response.nil?
       nil
@@ -86,7 +86,7 @@ class Network
   def post_logs(events, statsig_metadata)
     begin
       json_body = JSON.generate({'events' => events, 'statsigMetadata' => statsig_metadata})
-      post_helper('log_event', body: json_body, retries: 5)
+      post_helper('log_event', json_body, retries: 5)
     rescue
     end
   end

data/lib/statsig_driver.rb

@@ -56,9 +56,11 @@ class StatsigDriver
 
     if res == $fetch_from_server
       res = check_gate_fallback(user, gate_name)
+      # exposure logged by the server
+    else
+      @logger.log_gate_exposure(user, res.name, res.gate_value, res.rule_id)
     end
 
-    @logger.log_gate_exposure(user, res.name, res.gate_value, res.rule_id)
     res.gate_value
   end
 
@@ -80,11 +82,12 @@ class StatsigDriver
 
     if res == $fetch_from_server
       res = get_config_fallback(user, dynamic_config_name)
+      # exposure logged by the server
+    else
+      @logger.log_config_exposure(user, res.name, res.rule_id)
     end
 
-    result_config = DynamicConfig.new(res.name, res.json_value, res.rule_id)
-    @logger.log_config_exposure(user, result_config.name, result_config.rule_id)
-    result_config
+    DynamicConfig.new(res.name, res.json_value, res.rule_id)
   end
 
   def log_event(user, event_name, value = nil, metadata = nil)
@@ -96,7 +99,7 @@ class StatsigDriver
     user = normalize_user(user)
 
     event = StatsigEvent.new(event_name)
-    event.user = user&.serialize
+    event.user = user
     event.value = value
     event.metadata = metadata
     event.statsig_metadata = @statsig_metadata

data/lib/statsig_event.rb

@@ -1,15 +1,22 @@
 class StatsigEvent
   attr_accessor :value
-  attr_accessor :user
   attr_accessor :metadata
   attr_accessor :statsig_metadata
+  attr_reader :user
+
   def initialize(event_name)
     @event_name = event_name
     @time = Time.now.to_f * 1000
   end
 
+  def user=(value)
+    if value.is_a?(StatsigUser)
+      @user = value.serialize(true)
+    end
+  end
+
   def serialize
-    return {
+    {
       'eventName' => @event_name,
       'metadata' => @metadata,
       'value' => @value,

data/lib/statsig_logger.rb

@@ -54,8 +54,6 @@ class StatsigLogger
     flush_events = @events.map { |e| e.serialize() }
     @events = []
 
-    Thread.new do
-      @network.post_logs(flush_events, @statsig_metadata)
-    end
+    @network.post_logs(flush_events, @statsig_metadata)
   end
 end

data/lib/statsig_user.rb

@@ -7,6 +7,7 @@ class StatsigUser
   attr_accessor :locale
   attr_accessor :app_version
   attr_accessor :statsig_environment
+  attr_accessor :private_attributes
 
   def custom
     @custom
@@ -29,11 +30,12 @@ class StatsigUser
       @app_version = user_hash['appVersion'] || user_hash['app_version']
       @custom = user_hash['custom']
       @statsig_environment = user_hash['statsigEnvironment']
+      @private_attributes = user_hash['privateAttributes']
     end
   end
 
-  def serialize
-    {
+  def serialize(for_logging)
+    hash = {
       'userID' => @user_id,
       'email' => @email,
       'ip' => @ip,
@@ -43,7 +45,12 @@ class StatsigUser
       'appVersion' => @app_version,
       'custom' => @custom,
       'statsigEnvironment' => @statsig_environment,
+      'privateAttributes' => @private_attributes,
     }
+    if for_logging
+      hash.delete('privateAttributes')
+    end
+    hash
   end
 
   def value_lookup
@@ -62,6 +69,7 @@ class StatsigUser
       'appversion' => @app_version,
       'app_version' => @app_version,
       'custom' => @custom,
+      'privateAttributes' => @private_attributes,
     }
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: statsig
 version: !ruby/object:Gem::Version
-  version: 1.4.0
+  version: 1.6.2
 platform: ruby
 authors:
 - Statsig, Inc
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-08-13 00:00:00.000000000 Z
+date: 2021-09-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler