statsig 1.24.5 → 1.24.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1565b50f9e48bfcd6c456bc8e1681a9adbd523287f656b042c3cce803063d3b6
-  data.tar.gz: 8dbf5c5879173b3c44e0e155c8e6e649af5e0cb184e1fdd18a47cc8875fb32c0
+  metadata.gz: 29818b483f3950c841a00a05ee3007414cc759498d7656703049417c43cbbae5
+  data.tar.gz: 36c190613ab7fded21de40e82c238c3c8374e6efba502a6774752a15d796e07d
 SHA512:
-  metadata.gz: f8c6e02873f9274b49a42fe94f21d67eec6d2ba720841d107d1b0b3e7f39b3a208fc4bae478337829359e25d109f5e7409dffbd143e802ab5e1f1c2b0d08fcf0
-  data.tar.gz: f75079d1ba3837d66a8386aa106d2bb4f75e78371dcc53cf09e1921610841d81a958e351e1f1a2f5786ebf0c44c43215a709edcb1e21dd45b563f8b2f4f8004d
+  metadata.gz: 30e3d1941bb46e4eb21469f82d2f398e1380fc144b1076f28e521081a9de052bf96ad27821ca93c515e3e308ca8d793dff75a504435a792c008b4623effeb6a5
+  data.tar.gz: 80eb7090f03d1dcb2a8e78492c6d80fe33e364fea1446330ad30c17eb06dbe8486f7ab1d2776e53e9f3b058f7c4925d4d89ce1f08494e3fd814928c3ac4bd537

data/lib/evaluator.rb

@@ -447,16 +447,18 @@ module Statsig
       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
       when 'os_name', 'osname'
+        os = @ua_parser.parse_os(ua)
         return os&.family
       when 'os_version', 'osversion'
+        os = @ua_parser.parse_os(ua)
         return os&.version unless os&.version.nil?
       when 'browser_name', 'browsername'
+        parsed = @ua_parser.parse_ua(ua)
         return parsed.family
       when 'browser_version', 'browserversion'
+        parsed = @ua_parser.parse_ua(ua)
         return parsed.version.to_s
       else
         nil

data/lib/layer.rb

@@ -1,5 +1,6 @@
 # typed: false
 
+require 'sorbet-runtime'
 ##
 # Contains the current values from Statsig.
 # Will contain layer default values for all shared parameters in that layer.

data/lib/network.rb

@@ -33,6 +33,7 @@ module Statsig
       @local_mode = options.local_mode
       @timeout = options.network_timeout
       @backoff_multiplier = backoff_mult
+      @post_logs_retry_limit = options.post_logs_retry_limit
       @session_id = SecureRandom.uuid
     end
 
@@ -57,18 +58,19 @@ module Statsig
       if @timeout
         http = http.timeout(@timeout)
       end
+      backoff_adjusted = backoff > 10 ? backoff += Random.rand(10) : backoff # to deter overlap
       begin
         res = http.post(@api + endpoint, body: body)
       rescue StandardError => e
         ## network error retry
         return nil, e unless retries > 0
-        sleep backoff
+        sleep backoff_adjusted
         return post_helper(endpoint, body, retries - 1, backoff * @backoff_multiplier)
       end
       return res, nil if res.status.success?
       return nil, NetworkError.new("Got an exception when making request to #{@api + endpoint}: #{res.to_s}", res.status.to_i) unless retries > 0 && $retry_codes.include?(res.code)
       ## status code retry
-      sleep backoff
+      sleep backoff_adjusted
       post_helper(endpoint, body, retries - 1, backoff * @backoff_multiplier)
     end
 
@@ -97,7 +99,7 @@ module Statsig
     def post_logs(events)
       begin
         json_body = JSON.generate({ 'events' => events, 'statsigMetadata' => Statsig.get_statsig_metadata })
-        post_helper('log_event', json_body, 5)
+        post_helper('log_event', json_body, @post_logs_retry_limit)
       rescue
       end
     end

data/lib/statsig.rb

@@ -148,7 +148,7 @@ module Statsig
     @shared_instance&.get_layer(user, layer_name, StatsigDriver::GetLayerOptions.new(log_exposure: false))
   end
 
-  sig { params(user: StatsigUser, layer_name: String, parameter_name: String).returns(Layer) }
+  sig { params(user: StatsigUser, layer_name: String, parameter_name: String).void }
   ##
   # Logs an exposure event for the parameter in the given layer
   #
@@ -227,7 +227,7 @@ module Statsig
   def self.get_statsig_metadata
     {
       'sdkType' => 'ruby-server',
-      'sdkVersion' => '1.24.5',
+      'sdkVersion' => '1.24.6',
     }
   end
 

data/lib/statsig_driver.rb

@@ -170,7 +170,6 @@ class StatsigDriver
       event.user = user
       event.value = value
       event.metadata = metadata
-      event.statsig_metadata = Statsig.get_statsig_metadata
       @logger.log_event(event)
     })
   end
@@ -199,6 +198,7 @@ class StatsigDriver
   # @return [Hash]
   def get_client_initialize_response(user)
     @err_boundary.capture(-> {
+      validate_user(user)
       normalize_user(user)
       @evaluator.get_client_initialize_response(user)
     }, -> { nil })

data/lib/statsig_event.rb

@@ -1,9 +1,6 @@
 # typed: true
 class StatsigEvent
-  attr_accessor :value
-  attr_accessor :metadata
-  attr_accessor :statsig_metadata
-  attr_accessor :secondary_exposures
+  attr_accessor :value, :metadata, :statsig_metadata, :secondary_exposures
   attr_reader :user
 
   def initialize(event_name)
@@ -13,6 +10,7 @@ class StatsigEvent
     @secondary_exposures = nil
     @user = nil
     @time = (Time.now.to_f * 1000).to_i
+    @statsig_metadata = Statsig.get_statsig_metadata
   end
 
   def user=(value)

data/lib/statsig_logger.rb

@@ -16,10 +16,10 @@ module Statsig
 
       @logging_pool = Concurrent::ThreadPoolExecutor.new(
         min_threads: [2, Concurrent.processor_count].min,
-        max_threads: [2, Concurrent.processor_count].max,
+        max_threads: [5, Concurrent.processor_count].min,
         # max jobs pending before we start dropping
-        max_queue:   [2, Concurrent.processor_count].max * 5,
-        fallback_policy: :discard,
+        max_queue: [5, Concurrent.processor_count].min * 5,
+        fallback_policy: :discard
       )
 
       @background_flush = periodic_flush
@@ -44,7 +44,6 @@ module Statsig
       }
       return false if not is_unique_exposure(user, $gate_exposure_event, metadata)
       event.metadata = metadata
-      event.statsig_metadata = Statsig.get_statsig_metadata
 
       event.secondary_exposures = secondary_exposures.is_a?(Array) ? secondary_exposures : []
 
@@ -62,7 +61,6 @@ module Statsig
       }
       return false if not is_unique_exposure(user, $config_exposure_event, metadata)
       event.metadata = metadata
-      event.statsig_metadata = Statsig.get_statsig_metadata
       event.secondary_exposures = secondary_exposures.is_a?(Array) ? secondary_exposures : []
 
       safe_add_eval_details(eval_details, event)
@@ -90,7 +88,6 @@ module Statsig
       }
       return false if not is_unique_exposure(user, $layer_exposure_event, metadata)
       event.metadata = metadata
-      event.statsig_metadata = Statsig.get_statsig_metadata
       event.secondary_exposures = exposures.is_a?(Array) ? exposures : []
 
       safe_add_eval_details(config_evaluation.evaluation_details, event)

data/lib/statsig_options.rb

@@ -79,6 +79,10 @@ class StatsigOptions
   # Number of seconds before a network call is timed out
   attr_accessor :network_timeout
 
+  sig { returns(Integer) }
+  # Number of times to retry sending a batch of failed log events
+  attr_accessor :post_logs_retry_limit
+
   sig do
     params(
       environment: T.any(T::Hash[String, String], NilClass),
@@ -94,7 +98,8 @@ class StatsigOptions
       idlist_threadpool_size: Integer,
       disable_diagnostics_logging: T::Boolean,
       disable_sorbet_logging_handlers: T::Boolean,
-      network_timeout: T.any(Integer, NilClass)
+      network_timeout: T.any(Integer, NilClass),
+      post_logs_retry_limit: Integer
     ).void
   end
 
@@ -112,7 +117,8 @@ class StatsigOptions
     idlist_threadpool_size: 3,
     disable_diagnostics_logging: false,
     disable_sorbet_logging_handlers: false,
-    network_timeout: nil)
+    network_timeout: nil,
+    post_logs_retry_limit: 3)
     @environment = environment.is_a?(Hash) ? environment : nil
     @api_url_base = api_url_base
     @rulesets_sync_interval = rulesets_sync_interval
@@ -127,5 +133,6 @@ class StatsigOptions
     @disable_diagnostics_logging = disable_diagnostics_logging
     @disable_sorbet_logging_handlers = disable_sorbet_logging_handlers
     @network_timeout = network_timeout
+    @post_logs_retry_limit = post_logs_retry_limit
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: statsig
 version: !ruby/object:Gem::Version
-  version: 1.24.5
+  version: 1.24.6
 platform: ruby
 authors:
 - Statsig, Inc
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-04-03 00:00:00.000000000 Z
+date: 2023-05-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -44,14 +44,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '5.14'
+        version: 5.14.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '5.14'
+        version: 5.14.0
 - !ruby/object:Gem::Dependency
   name: spy
   requirement: !ruby/object:Gem::Requirement
@@ -123,19 +123,89 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '6.0'
 - !ruby/object:Gem::Dependency
-  name: user_agent_parser
+  name: rubocop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.28.2
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.28.2
+- !ruby/object:Gem::Dependency
+  name: parallel_tests
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '2.7'
-  type: :runtime
+  type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '2.7'
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.21'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.21'
+- !ruby/object:Gem::Dependency
+  name: simplecov-lcov
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.7.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.7.0
+- !ruby/object:Gem::Dependency
+  name: simplecov-cobertura
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.1'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.1'
+- !ruby/object:Gem::Dependency
+  name: user_agent_parser
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 2.15.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 2.15.0
 - !ruby/object:Gem::Dependency
   name: http
   requirement: !ruby/object:Gem::Requirement