statsig 1.6.0 → 1.7.0.beta.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a1985e842f0301387596fbd3d8db298ce485c013a502c09a1aff00916882afc0
-  data.tar.gz: 9cf814e0def7dc3fefd2db9d80f102ec318a41703e6638f66b8a8d844c751959
+  metadata.gz: cf5fda9f980ecae86129f6ea0bd4052b6af45cba14c83e38062b495cd06cfeb1
+  data.tar.gz: 5b1a231a8af145fec2303fc89998f3e9acf05323cbc55d9fea3850ba841ecd1e
 SHA512:
-  metadata.gz: ba2661bcd29ca31b3c458f8046add904029b03ecc0eeeb0c358760467389e58c5f48b5bc39456df1ea2e30051df951359a37e1045283a5d77f911291b0f7bfcd
-  data.tar.gz: 2631c284b8b792101558e90689353c79aa043a22cbf2f995f045efc4333f74d3d80fcca639ece7f11d734cc4a39b459edd08ce14f6603f86f09ad57afadf2c08
+  metadata.gz: 7f1c7cf9a5a549021f02c16bac3d1095143ee9231a3f4165db9040efbf9b16cef083b6955dc5f748dd5fcfe12953064e8c48874a1eb8f477c24f61d061a57890
+  data.tar.gz: a5f291bd14edb61286416c293eae1c14f3c519dc1c6548356fe7147667cc33ed6b327488412e2f0b5516139d71d06bacf0db3f6ee907685be9c7e8ca2cc6b119

data/lib/config_result.rb

@@ -3,11 +3,13 @@ class ConfigResult
   attr_accessor :gate_value
   attr_accessor :json_value
   attr_accessor :rule_id
+  attr_accessor :secondary_exposures
 
-  def initialize(name, gate_value = false, json_value = {}, rule_id = '')
+  def initialize(name, gate_value = false, json_value = {}, rule_id = '', secondary_exposures = [])
     @name = name
     @gate_value = gate_value
     @json_value = json_value
     @rule_id = rule_id
+    @secondary_exposures = secondary_exposures.is_a?(Array) ? secondary_exposures : []
   end
 end

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

@@ -31,37 +31,53 @@ class Evaluator
   private
 
   def eval_spec(user, config)
+    default_rule_id = 'default'
+    exposures = []
     if config['enabled']
       i = 0
       until i >= config['rules'].length do
         rule = config['rules'][i]
         result = self.eval_rule(user, rule)
         return $fetch_from_server if result == $fetch_from_server
-        if result
+        exposures = exposures + result['exposures'] if result['exposures'].is_a? Array
+        if result['value']
           pass = self.eval_pass_percent(user, rule, config['salt'])
           return ConfigResult.new(
             config['name'],
             pass,
             pass ? rule['returnValue'] : config['defaultValue'],
             rule['id'],
+            exposures
           )
         end
 
         i += 1
       end
+    elsif (default_rule_id = 'disabled')
     end
 
-    ConfigResult.new(config['name'], false, config['defaultValue'], 'default')
+    ConfigResult.new(config['name'], false, config['defaultValue'], default_rule_id, exposures)
   end
 
   def eval_rule(user, rule)
+    exposures = []
+    pass = true
     i = 0
     until i >= rule['conditions'].length do
       result = self.eval_condition(user, rule['conditions'][i])
-      return result unless result == true
+      if result == $fetch_from_server
+        return $fetch_from_server
+      end
+
+      if result.is_a?(Hash)
+        exposures = exposures + result['exposures'] if result['exposures'].is_a? Array
+        pass = false if result['value'] == false
+      elsif result == false
+        pass = false
+      end
       i += 1
     end
-    true
+    { 'value' => pass, 'exposures' => exposures }
   end
 
   def eval_condition(user, condition)
@@ -82,7 +98,18 @@ class Evaluator
     when 'fail_gate', 'pass_gate'
       other_gate_result = self.check_gate(user, target)
       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
+
+      gate_value = other_gate_result&.gate_value == true
+      new_exposure = {
+        'gate' => target,
+        'gateValue' => gate_value ? 'true' : 'false',
+        'ruleID' => other_gate_result&.rule_id
+      }
+      exposures = other_gate_result&.secondary_exposures&.append(new_exposure)
+      return {
+        'value' => type == 'pass_gate' ? gate_value : !gate_value,
+        'exposures' => exposures
+      }
     when 'ip_based'
       value = get_value_from_user(user, field) || get_value_from_ip(user, field)
       return $fetch_from_server if value == $fetch_from_server
@@ -138,13 +165,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'

data/lib/network.rb

@@ -24,14 +24,14 @@ class Network
       }).accept(:json)
     begin
       res = http.post(@api + endpoint, body: body)
-    rescue
+    rescue StandardError => e
       ## network error retry
-      return nil unless retries > 0
+      return nil, e unless retries > 0
       sleep backoff
       return post_helper(endpoint, body, retries - 1, backoff * @backoff_multiplier)
     end
-    return res unless !res.status.success?
-    return nil unless retries > 0 && $retry_codes.include?(res.code)
+    return res, nil unless !res.status.success?
+    return nil, StandardError.new("Got an exception when making request to #{@api + endpoint}: #{res.to_s}") unless retries > 0 && $retry_codes.include?(res.code)
     ## status code retry
     sleep backoff
     post_helper(endpoint, body, retries - 1, backoff * @backoff_multiplier)
@@ -40,7 +40,7 @@ class Network
   def check_gate(user, gate_name)
     begin
       request_body = JSON.generate({'user' => user&.serialize(false), 'gateName' => gate_name})
-      response = post_helper('check_gate', request_body)
+      response, _ = post_helper('check_gate', request_body)
       return JSON.parse(response.body) unless response.nil?
       false
     rescue
@@ -51,7 +51,7 @@ class Network
   def get_config(user, dynamic_config_name)
     begin
       request_body = JSON.generate({'user' => user&.serialize(false), 'configName' => dynamic_config_name})
-      response = post_helper('get_config', request_body)
+      response, _ = post_helper('get_config', request_body)
       return JSON.parse(response.body) unless response.nil?
       nil
     rescue
@@ -61,13 +61,13 @@ class Network
 
   def download_config_specs
     begin
-      response = post_helper('download_config_specs', JSON.generate({'sinceTime' => @last_sync_time}))
-      return nil unless !response.nil?
+      response, e = post_helper('download_config_specs', JSON.generate({'sinceTime' => @last_sync_time}))
+      return nil, e if response.nil?
       json_body = JSON.parse(response.body)
       @last_sync_time = json_body['time']
-      return json_body
-    rescue
-      return nil
+      return json_body, nil
+    rescue StandardError => e
+      return nil, e
     end
   end
 
@@ -75,7 +75,7 @@ class Network
     Thread.new do
       loop do
         sleep 10
-        specs = download_config_specs
+        specs, _ = download_config_specs
         unless specs.nil?
           callback.call(specs)
         end
@@ -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.rb

@@ -1,13 +1,13 @@
 require 'statsig_driver'
 
 module Statsig
-  def self.initialize(secret_key, options = nil)
+  def self.initialize(secret_key, options = nil, error_callback = nil)
     unless @shared_instance.nil?
       puts 'Statsig already initialized.'
       return @shared_instance
     end
 
-    @shared_instance = StatsigDriver.new(secret_key, options)
+    @shared_instance = StatsigDriver.new(secret_key, options, error_callback)
   end
 
   def self.check_gate(user, gate_name)

data/lib/statsig_driver.rb

@@ -8,7 +8,7 @@ require 'statsig_user'
 require 'spec_store'
 
 class StatsigDriver
-  def initialize(secret_key, options = nil)
+  def initialize(secret_key, options = nil, error_callback = nil)
     super()
     if !secret_key.is_a?(String) || !secret_key.start_with?('secret-')
       raise 'Invalid secret key provided. Provide your project secret key from the Statsig console'
@@ -17,7 +17,7 @@ class StatsigDriver
       raise 'Invalid options provided. Either provide a valid StatsigOptions object or nil'
     end
 
-    @options = options || StatsigOptions.new()
+    @options = options || StatsigOptions.new
     @shutdown = false
     @secret_key = secret_key
     @net = Network.new(secret_key, @options.api_url_base)
@@ -27,7 +27,7 @@ class StatsigDriver
     }
     @logger = StatsigLogger.new(@net, @statsig_metadata)
 
-    downloaded_specs = @net.download_config_specs
+    downloaded_specs, e = @net.download_config_specs
     unless downloaded_specs.nil?
       @initialized = true
     end
@@ -36,6 +36,8 @@ class StatsigDriver
     @evaluator = Evaluator.new(@store)
 
     @polling_thread = @net.poll_for_changes(-> (config_specs) { @store.process(config_specs) })
+
+    error_callback.call(e) unless error_callback.nil?
   end
 
   def check_gate(user, gate_name)
@@ -58,7 +60,7 @@ class StatsigDriver
       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)
+      @logger.log_gate_exposure(user, res.name, res.gate_value, res.rule_id, res.secondary_exposures)
     end
 
     res.gate_value
@@ -84,12 +86,19 @@ class StatsigDriver
       res = get_config_fallback(user, dynamic_config_name)
       # exposure logged by the server
     else
-      @logger.log_config_exposure(user, res.name, res.rule_id)
+      @logger.log_config_exposure(user, res.name, res.rule_id, res.secondary_exposures)
     end
 
     DynamicConfig.new(res.name, res.json_value, res.rule_id)
   end
 
+  def get_experiment(user, experiment_name)
+    if !experiment_name.is_a?(String) || experiment_name.empty?
+      raise "Invalid experiment_name provided"
+    end
+    get_config(user, experiment_name)
+  end
+
   def log_event(user, event_name, value = nil, metadata = nil)
     if !user.nil? && !user.instance_of?(StatsigUser)
       raise 'Must provide a valid StatsigUser or nil'

data/lib/statsig_event.rb

@@ -2,6 +2,7 @@ class StatsigEvent
   attr_accessor :value
   attr_accessor :metadata
   attr_accessor :statsig_metadata
+  attr_accessor :secondary_exposures
   attr_reader :user
 
   def initialize(event_name)
@@ -23,6 +24,7 @@ class StatsigEvent
       'user' => @user,
       'time' => @time,
       'statsigMetadata' => @statsig_metadata,
+      'secondaryExposures' => @secondary_exposures
     }
   end
 end

data/lib/statsig_logger.rb

@@ -21,7 +21,7 @@ class StatsigLogger
     end
   end
 
-  def log_gate_exposure(user, gate_name, value, rule_id)
+  def log_gate_exposure(user, gate_name, value, rule_id, secondary_exposures)
     event = StatsigEvent.new($gate_exposure_event)
     event.user = user
     event.metadata = {
@@ -30,10 +30,11 @@ class StatsigLogger
       'ruleID' => rule_id
     }
     event.statsig_metadata = @statsig_metadata
+    event.secondary_exposures = secondary_exposures.is_a?(Array) ? secondary_exposures : []
     log_event(event)
   end
 
-  def log_config_exposure(user, config_name, rule_id)
+  def log_config_exposure(user, config_name, rule_id, secondary_exposures)
     event = StatsigEvent.new($config_exposure_event)
     event.user = user
     event.metadata = {
@@ -41,6 +42,7 @@ class StatsigLogger
       'ruleID' => rule_id
     }
     event.statsig_metadata = @statsig_metadata
+    event.secondary_exposures = secondary_exposures.is_a?(Array) ? secondary_exposures : []
     log_event(event)
   end
 
@@ -51,11 +53,9 @@ class StatsigLogger
     if @events.length == 0
       return
     end
-    flush_events = @events.map { |e| e.serialize() }
+    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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: statsig
 version: !ruby/object:Gem::Version
-  version: 1.6.0
+  version: 1.7.0.beta.1
 platform: ruby
 authors:
 - Statsig, Inc
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-09-09 00:00:00.000000000 Z
+date: 2021-10-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -141,9 +141,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - ">"
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 1.3.1
 requirements: []
 rubygems_version: 3.2.3
 signing_key: