statsig 1.2.0 → 1.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 90f13c55a79aa919b94e633f69d39b3cb997a92cceec3e27bdb3e00e00e8e3ef
-  data.tar.gz: cd40c863516da1b3ff0730959ccbc3fa664afe0e613775f0369a1377ccc06123
+  metadata.gz: 3ac164a57fadf37c1c9ce2b978df30da43901cc3b53e03e1b3e52a72e19981cb
+  data.tar.gz: aad0387cd6a80bfd3cbf501118466f444790ec387b45a96d562a3256331ae034
 SHA512:
-  metadata.gz: a4c73b125853c8128d5aa57a60acfe1e80a0aa8ffc9f1fdcd89ea74e9a6ac033c93262103850d0c13ec84052ca9b9a464627836a1af945e6100c036c0fbc1359
-  data.tar.gz: 86bf2fe76cdc5cdc29126bd188a33e746f2a143e56fce1cd2e7d5a242622acb4b3783694f01e11a6e8ce10c707bc38a57a055694e15c965ee584bb951c6acb0d
+  metadata.gz: f7eeefd82a45217becadee9783d2616de5e6997d29236fecea526947a34e54f921125fbf018f57a1fbf2f7560c00d80721f632a9498e0f065b6914b035beef5f
+  data.tar.gz: 24b8c55535b7656109b669d898351e536953c83ecc6858e94eabacbdb36f8c9e600b55bb9cd06fc8b043a84305f351e645c07a673a85c1688668de6181c125f3

data/lib/dynamic_config.rb

@@ -9,8 +9,8 @@ class DynamicConfig
     @rule_id = rule_id
   end
 
-  def get(index)
-    return nil if @value.nil?
-    value[index]
+  def get(index, default_value)
+    return default_value if @value.nil? || !@value.key?(index)
+    @value[index]
   end
 end

data/lib/evaluator.rb

@@ -79,8 +79,7 @@ class Evaluator
     case type
     when 'public'
       return true
-    when 'fail_gate'
-    when 'pass_gate'
+    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
@@ -109,10 +108,7 @@ class Evaluator
       return $fetch_from_server
     end
 
-    return $fetch_from_server if value == $fetch_from_server
-    return false if value.nil?
-
-    return $fetch_from_server unless operator.is_a?(String)
+    return $fetch_from_server if value == $fetch_from_server || !operator.is_a?(String)
     operator = operator.downcase
 
     case operator

data/lib/network.rb

@@ -2,8 +2,10 @@ require 'http'
 require 'json'
 require 'dynamic_config'
 
+$retry_codes = [408, 500, 502, 503, 504, 522, 524, 599]
+
 class Network
-  def initialize(server_secret, api)
+  def initialize(server_secret, api, backoff_mult = 10)
     super()
     unless api.end_with?('/')
       api += '/'
@@ -11,22 +13,36 @@ class Network
     @server_secret = server_secret
     @api = api
     @last_sync_time = 0
+    @backoff_multiplier = backoff_mult
   end
 
-  def post_helper(endpoint, body)
+  def post_helper(endpoint, body, retries = 0, backoff = 1)
     http = HTTP.headers(
       {"STATSIG-API-KEY" => @server_secret,
        "STATSIG-CLIENT-TIME" => (Time.now.to_f * 1000).to_s,
        "Content-Type" => "application/json; charset=UTF-8"
       }).accept(:json)
-    http.post(@api + endpoint, body: body)
+    begin
+      res = http.post(@api + endpoint, body: body)
+    rescue
+      ## network error retry
+      return nil 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)
+    ## status code retry
+    sleep backoff
+    post_helper(endpoint, body, retries - 1, backoff * @backoff_multiplier)
   end
 
   def check_gate(user, gate_name)
     begin
       request_body = JSON.generate({'user' => user&.serialize, 'gateName' => gate_name})
       response = post_helper('check_gate', request_body)
-      return JSON.parse(response.body)
+      return JSON.parse(response.body) unless response.nil?
+      false
     rescue
       return false
     end
@@ -36,7 +52,8 @@ class Network
     begin
       request_body = JSON.generate({'user' => user&.serialize, 'configName' => dynamic_config_name})
       response = post_helper('get_config', request_body)
-      return JSON.parse(response.body)
+      return JSON.parse(response.body) unless response.nil?
+      nil
     rescue
       return nil
     end
@@ -45,6 +62,7 @@ class Network
   def download_config_specs
     begin
       response = post_helper('download_config_specs', JSON.generate({'sinceTime' => @last_sync_time}))
+      return nil unless !response.nil?
       json_body = JSON.parse(response.body)
       @last_sync_time = json_body['time']
       return json_body
@@ -68,9 +86,8 @@ 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)
+      post_helper('log_event', body: json_body, retries: 5)
     rescue
-      # TODO: retries
     end
   end
 end

data/lib/statsig_driver.rb

@@ -105,7 +105,7 @@ class StatsigDriver
 
   def shutdown
     @shutdown = true
-    @logger.flush
+    @logger.flush(true)
     @polling_thread&.exit
   end
 

data/lib/statsig_logger.rb

@@ -8,6 +8,10 @@ class StatsigLogger
     @network = network
     @statsig_metadata = statsig_metadata
     @events = []
+    @background_flush = Thread.new do
+      sleep 60
+      flush
+    end
   end
 
   def log_event(event)
@@ -40,13 +44,18 @@ class StatsigLogger
     log_event(event)
   end
 
-  def flush
+  def flush(closing = false)
+    if closing
+      @background_flush.exit
+    end
     if @events.length == 0
       return
     end
     flush_events = @events.map { |e| e.serialize() }
     @events = []
 
-    @network.post_logs(flush_events, @statsig_metadata)
+    Thread.new do
+      @network.post_logs(flush_events, @statsig_metadata)
+    end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: statsig
 version: !ruby/object:Gem::Version
-  version: 1.2.0
+  version: 1.4.0
 platform: ruby
 authors:
 - Statsig, Inc
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-07-31 00:00:00.000000000 Z
+date: 2021-08-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -24,6 +24,48 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '2.1'
+- !ruby/object:Gem::Dependency
+  name: webmock
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.13'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.13'
+- !ruby/object:Gem::Dependency
+  name: minitest
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '5.14'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '5.14'
+- !ruby/object:Gem::Dependency
+  name: spy
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.0'
 - !ruby/object:Gem::Dependency
   name: user_agent_parser
   requirement: !ruby/object:Gem::Requirement