splitclient-rb 5.1.0 → 5.1.1.pre.rc1

data/lib/splitclient-rb/cache/senders/metrics_sender.rb

@@ -2,9 +2,8 @@ module SplitIoClient
   module Cache
     module Senders
       class MetricsSender
-        def initialize(metrics_repository, config, api_key)
+        def initialize(metrics_repository, api_key)
           @metrics_repository = metrics_repository
-          @config = config
           @api_key = api_key
         end
 
@@ -23,13 +22,13 @@ module SplitIoClient
         private
 
         def metrics_thread
-          @config.threads[:metrics_sender] = Thread.new do
-            @config.logger.info('Starting metrics service')
+          SplitIoClient.configuration.threads[:metrics_sender] = Thread.new do
+            SplitIoClient.configuration.logger.info('Starting metrics service')
 
             loop do
               post_metrics
 
-              sleep(SplitIoClient::Utilities.randomize_interval(@config.metrics_refresh_rate))
+              sleep(SplitIoClient::Utilities.randomize_interval(SplitIoClient.configuration.metrics_refresh_rate))
             end
           end
         end
@@ -37,11 +36,11 @@ module SplitIoClient
         def post_metrics
           metrics_client.post
         rescue StandardError => error
-          @config.log_found_exception(__method__.to_s, error)
+          SplitIoClient.configuration.log_found_exception(__method__.to_s, error)
         end
 
         def metrics_client
-          SplitIoClient::Api::Metrics.new(@api_key, @config, @metrics_repository)
+          SplitIoClient::Api::Metrics.new(@api_key, @metrics_repository)
         end
       end
     end

data/lib/splitclient-rb/cache/stores/sdk_blocker.rb

@@ -8,8 +8,7 @@ module SplitIoClient
         attr_reader :splits_repository
         attr_writer :splits_thread, :segments_thread
 
-        def initialize(config, splits_repository, segments_repository)
-          @config = config
+        def initialize(splits_repository, segments_repository)
           @splits_repository = splits_repository
           @segments_repository = segments_repository
 
@@ -27,14 +26,14 @@ module SplitIoClient
 
         def block
           begin
-            Timeout::timeout(@config.block_until_ready) do
+            Timeout::timeout(SplitIoClient.configuration.block_until_ready) do
               sleep 0.1 until ready?
             end
           rescue Timeout::Error
             fail SDKBlockerTimeoutExpiredException, 'SDK start up timeout expired'
           end
 
-          @config.logger.info('SplitIO SDK is ready')
+          SplitIoClient.configuration.logger.info('SplitIO SDK is ready')
           @splits_thread.run
           @segments_thread.run
         end

data/lib/splitclient-rb/cache/stores/segment_store.rb

@@ -4,9 +4,8 @@ module SplitIoClient
       class SegmentStore
         attr_reader :segments_repository
 
-        def initialize(segments_repository, config, api_key, metrics, sdk_blocker = nil)
+        def initialize(segments_repository, api_key, metrics, sdk_blocker = nil)
           @segments_repository = segments_repository
-          @config = config
           @api_key = api_key
           @metrics = metrics
           @sdk_blocker = sdk_blocker
@@ -29,9 +28,9 @@ module SplitIoClient
         private
 
         def segments_thread
-          @config.threads[:segment_store] = @sdk_blocker.segments_thread = Thread.new do
-            @config.logger.info('Starting segments fetcher service')
-            @config.block_until_ready > 0 ? blocked_store : unblocked_store
+          SplitIoClient.configuration.threads[:segment_store] = @sdk_blocker.segments_thread = Thread.new do
+            SplitIoClient.configuration.logger.info('Starting segments fetcher service')
+            SplitIoClient.configuration.block_until_ready > 0 ? blocked_store : unblocked_store
           end
         end
 
@@ -40,15 +39,15 @@ module SplitIoClient
             next unless @sdk_blocker.splits_repository.ready?
 
             store_segments
-            @config.logger.debug("Segment names: #{@segments_repository.used_segment_names.to_a}") if @config.debug_enabled
+            SplitIoClient.configuration.logger.debug("Segment names: #{@segments_repository.used_segment_names.to_a}") if SplitIoClient.configuration.debug_enabled
 
             unless @sdk_blocker.ready?
               @sdk_blocker.segments_ready!
-              @config.logger.info('segments are ready')
+              SplitIoClient.configuration.logger.info('segments are ready')
             end
 
-            sleep_for = random_interval(@config.segments_refresh_rate)
-            @config.logger.debug("Segments store is sleeping for: #{sleep_for} seconds") if @config.debug_enabled
+            sleep_for = random_interval(SplitIoClient.configuration.segments_refresh_rate)
+            SplitIoClient.configuration.logger.debug("Segments store is sleeping for: #{sleep_for} seconds") if SplitIoClient.configuration.debug_enabled
             sleep(sleep_for)
           end
         end
@@ -57,14 +56,14 @@ module SplitIoClient
           loop do
             store_segments
 
-            sleep(random_interval(@config.segments_refresh_rate))
+            sleep(random_interval(SplitIoClient.configuration.segments_refresh_rate))
           end
         end
 
         def store_segments
           segments_api.store_segments_by_names(@segments_repository.used_segment_names)
         rescue StandardError => error
-          @config.log_found_exception(__method__.to_s, error)
+          SplitIoClient.configuration.log_found_exception(__method__.to_s, error)
         end
 
         def random_interval(interval)
@@ -74,7 +73,7 @@ module SplitIoClient
         end
 
         def segments_api
-          SplitIoClient::Api::Segments.new(@api_key, @config, @metrics, @segments_repository)
+          SplitIoClient::Api::Segments.new(@api_key, @metrics, @segments_repository)
         end
       end
     end

data/lib/splitclient-rb/cache/stores/split_store.rb

@@ -4,9 +4,8 @@ module SplitIoClient
       class SplitStore
         attr_reader :splits_repository
 
-        def initialize(splits_repository, config, api_key, metrics, sdk_blocker = nil)
+        def initialize(splits_repository, api_key, metrics, sdk_blocker = nil)
           @splits_repository = splits_repository
-          @config = config
           @api_key = api_key
           @metrics = metrics
           @sdk_blocker = sdk_blocker
@@ -29,12 +28,12 @@ module SplitIoClient
         private
 
         def splits_thread
-          @config.threads[:split_store] = @sdk_blocker.splits_thread = Thread.new do
-            @config.logger.info('Starting splits fetcher service')
+          SplitIoClient.configuration.threads[:split_store] = @sdk_blocker.splits_thread = Thread.new do
+            SplitIoClient.configuration.logger.info('Starting splits fetcher service')
             loop do
               store_splits
 
-              sleep(random_interval(@config.features_refresh_rate))
+              sleep(random_interval(SplitIoClient.configuration.features_refresh_rate))
             end
           end
         end
@@ -49,15 +48,15 @@ module SplitIoClient
           @splits_repository.set_segment_names(data[:segment_names])
           @splits_repository.set_change_number(data[:till])
 
-          @config.logger.debug("segments seen(#{data[:segment_names].length}): #{data[:segment_names].to_a}") if @config.debug_enabled
+          SplitIoClient.configuration.logger.debug("segments seen(#{data[:segment_names].length}): #{data[:segment_names].to_a}") if SplitIoClient.configuration.debug_enabled
 
-          if @config.block_until_ready > 0 && !@sdk_blocker.ready?
+          if SplitIoClient.configuration.block_until_ready > 0 && !@sdk_blocker.ready?
             @sdk_blocker.splits_ready!
-            @config.logger.info('splits are ready')
+            SplitIoClient.configuration.logger.info('splits are ready')
           end
 
         rescue StandardError => error
-          @config.log_found_exception(__method__.to_s, error)
+          SplitIoClient.configuration.log_found_exception(__method__.to_s, error)
         end
 
         def random_interval(interval)
@@ -67,12 +66,12 @@ module SplitIoClient
         end
 
         def splits_since(since)
-          SplitIoClient::Api::Splits.new(@api_key, @config, @metrics).since(since)
+          SplitIoClient::Api::Splits.new(@api_key, @metrics).since(since)
         end
 
         def add_split_unless_archived(split)
           if Engine::Models::Split.archived?(split)
-            @config.logger.debug("Seeing archived split #{split[:name]}") if @config.debug_enabled
+            SplitIoClient.configuration.logger.debug("Seeing archived split #{split[:name]}") if SplitIoClient.configuration.debug_enabled
 
             remove_archived_split(split)
           else
@@ -81,13 +80,13 @@ module SplitIoClient
         end
 
         def remove_archived_split(split)
-          @config.logger.debug("removing split from store(#{split})") if @config.debug_enabled
+          SplitIoClient.configuration.logger.debug("removing split from store(#{split})") if SplitIoClient.configuration.debug_enabled
 
           @splits_repository.remove_split(split[:name])
         end
 
         def store_split(split)
-          @config.logger.debug("storing split (#{split[:name]})") if @config.debug_enabled
+          SplitIoClient.configuration.logger.debug("storing split (#{split[:name]})") if SplitIoClient.configuration.debug_enabled
 
           @splits_repository.add_split(split)
         end

data/lib/splitclient-rb/clients/localhost_split_client.rb

@@ -42,12 +42,12 @@ module SplitIoClient
     # @return [Treatment]  treatment constant value
     def get_treatment(id, feature, attributes = nil)
       unless id
-        @config.logger.warn('id was null for feature: ' + feature)
+        SplitIoClient.configuration.logger.warn('id was null for feature: ' + feature)
         return SplitIoClient::Engine::Models::Treatment::CONTROL
       end
 
       unless feature
-        @config.logger.warn('feature was null for id: ' + id)
+        SplitIoClient.configuration.logger.warn('feature was null for id: ' + id)
         return SplitIoClient::Engine::Models::Treatment::CONTROL
       end
 

data/lib/splitclient-rb/clients/split_client.rb

@@ -6,9 +6,7 @@ module SplitIoClient
     # @param api_key [String] the API key for your split account
     #
     # @return [SplitIoClient] split.io client instance
-    def initialize(api_key, config = {}, adapter = nil, splits_repository, segments_repository, impressions_repository, metrics_repository, events_repository)
-      @config = config
-
+    def initialize(api_key, adapter = nil, splits_repository, segments_repository, impressions_repository, metrics_repository, events_repository)
       @splits_repository = splits_repository
       @segments_repository = segments_repository
       @impressions_repository = impressions_repository
@@ -27,7 +25,7 @@ module SplitIoClient
           memo.merge!(name => get_treatment(key, name, attributes, data, false, true, evaluator))
         end
 
-      unless @config.disable_impressions
+      unless SplitIoClient.configuration.disable_impressions
         time = (Time.now.to_f * 1000.0).to_i
         @impressions_repository.add_bulk(
           matching_key, bucketing_key, treatments_labels_change_numbers, time
@@ -63,12 +61,12 @@ module SplitIoClient
       evaluator ||= Engine::Parser::Evaluator.new(@segments_repository, @splits_repository)
 
       if matching_key.nil?
-        @config.logger.warn('matching_key was null for split_name: ' + split_name.to_s)
+        SplitIoClient.configuration.logger.warn('matching_key was null for split_name: ' + split_name.to_s)
         return parsed_treatment(multiple, treatment_data)
       end
 
       if split_name.nil?
-        @config.logger.warn('split_name was null for key: ' + key)
+        SplitIoClient.configuration.logger.warn('split_name was null for key: ' + key)
         return parsed_treatment(multiple, treatment_data)
       end
 
@@ -78,7 +76,7 @@ module SplitIoClient
         split = multiple ? split_data : @splits_repository.get_split(split_name)
 
         if split.nil?
-          @config.logger.debug("split_name: #{split_name} does not exist. Returning CONTROL")
+          SplitIoClient.configuration.logger.debug("split_name: #{split_name} does not exist. Returning CONTROL")
           return parsed_treatment(multiple, treatment_data)
         else
           treatment_data =
@@ -87,7 +85,7 @@ module SplitIoClient
           )
         end
       rescue StandardError => error
-        @config.log_found_exception(__method__.to_s, error)
+        SplitIoClient.configuration.log_found_exception(__method__.to_s, error)
 
         store_impression(
           split_name, matching_key, bucketing_key,
@@ -108,7 +106,7 @@ module SplitIoClient
         # Measure
         @adapter.metrics.time('sdk.get_treatment', latency)
       rescue StandardError => error
-        @config.log_found_exception(__method__.to_s, error)
+        SplitIoClient.configuration.log_found_exception(__method__.to_s, error)
 
         store_impression(
           split_name, matching_key, bucketing_key,
@@ -126,10 +124,10 @@ module SplitIoClient
     end
 
     def destroy
-      @config.logger.info('Split client shutdown started...') if @config.debug_enabled
+      SplitIoClient.configuration.logger.info('Split client shutdown started...') if SplitIoClient.configuration.debug_enabled
 
-      @config.threads[:impressions_sender].raise(SplitIoClient::ImpressionShutdownException)
-      @config.threads.reject { |k, _| k == :impressions_sender }.each do |name, thread|
+      SplitIoClient.configuration.threads[:impressions_sender].raise(SplitIoClient::ImpressionShutdownException)
+      SplitIoClient.configuration.threads.reject { |k, _| k == :impressions_sender }.each do |name, thread|
         Thread.kill(thread)
       end
 
@@ -138,19 +136,19 @@ module SplitIoClient
       @segments_repository.clear
       @events_repository.clear
 
-      @config.logger.info('Split client shutdown complete') if @config.debug_enabled
+      SplitIoClient.configuration.logger.info('Split client shutdown complete') if SplitIoClient.configuration.debug_enabled
     end
 
     def store_impression(split_name, matching_key, bucketing_key, treatment, store_impressions, attributes)
       time = (Time.now.to_f * 1000.0).to_i
 
-      return if @config.disable_impressions || !store_impressions
+      return if SplitIoClient.configuration.disable_impressions || !store_impressions
 
       @impressions_repository.add(split_name,
         'keyName' => matching_key,
         'bucketingKey' => bucketing_key,
         'treatment' => treatment[:treatment],
-        'label' => @config.labels_enabled ? treatment[:label] : nil,
+        'label' => SplitIoClient.configuration.labels_enabled ? treatment[:label] : nil,
         'time' => time,
         'changeNumber' => treatment[:change_number]
       )
@@ -158,7 +156,7 @@ module SplitIoClient
       route_impression(split_name, matching_key, bucketing_key, time, treatment, attributes)
 
     rescue StandardError => error
-      @config.log_found_exception(__method__.to_s, error)
+      SplitIoClient.configuration.log_found_exception(__method__.to_s, error)
     end
 
     def route_impression(split_name, matching_key, bucketing_key, time, treatment, attributes)
@@ -184,7 +182,7 @@ module SplitIoClient
     end
 
     def impression_router
-      @impression_router ||= SplitIoClient::ImpressionRouter.new(@config)
+      @impression_router ||= SplitIoClient::ImpressionRouter.new
     end
 
     def track(key, traffic_type, event_type, value = nil)

data/lib/splitclient-rb/engine/api/client.rb

@@ -5,39 +5,39 @@ module SplitIoClient
     class Client
       RUBY_ENCODING = '1.9'.respond_to?(:force_encoding)
 
-      def get_api(url, config, api_key, params = {})
+      def get_api(url, api_key, params = {})
         api_client.get(url, params) do |req|
-          req.headers = common_headers(api_key, config).merge('Accept-Encoding' => 'gzip')
+          req.headers = common_headers(api_key).merge('Accept-Encoding' => 'gzip')
 
-          req.options[:timeout] = config.read_timeout
-          req.options[:open_timeout] = config.connection_timeout
+          req.options[:timeout] = SplitIoClient.configuration.read_timeout
+          req.options[:open_timeout] = SplitIoClient.configuration.connection_timeout
 
-          config.logger.debug("GET #{url} proxy: #{api_client.proxy}") if config.debug_enabled
+          SplitIoClient.configuration.logger.debug("GET #{url} proxy: #{api_client.proxy}") if SplitIoClient.configuration.debug_enabled
         end
       rescue StandardError => e
-        config.logger.warn("#{e}\nURL:#{url}\nparams:#{params}")
+        SplitIoClient.configuration.logger.warn("#{e}\nURL:#{url}\nparams:#{params}")
         raise 'Split SDK failed to connect to backend to retrieve information'
       end
 
-      def post_api(url, config, api_key, data, headers = {}, params = {})
+      def post_api(url, api_key, data, headers = {}, params = {})
         api_client.post(url) do |req|
-          req.headers = common_headers(api_key, config)
+          req.headers = common_headers(api_key)
             .merge('Content-Type' => 'application/json')
             .merge(headers)
 
           req.body = data.to_json
 
-          req.options[:timeout] = config.read_timeout
-          req.options[:open_timeout] = config.connection_timeout
+          req.options[:timeout] = SplitIoClient.configuration.read_timeout
+          req.options[:open_timeout] = SplitIoClient.configuration.connection_timeout
 
-          if config.transport_debug_enabled
-            config.logger.debug("POST #{url} #{req.body}")
-          elsif config.debug_enabled
-            config.logger.debug("POST #{url}")
+          if SplitIoClient.configuration.transport_debug_enabled
+            SplitIoClient.configuration.logger.debug("POST #{url} #{req.body}")
+          elsif SplitIoClient.configuration.debug_enabled
+            SplitIoClient.configuration.logger.debug("POST #{url}")
           end
         end
       rescue StandardError => e
-        config.logger.warn("#{e}\nURL:#{url}\ndata:#{data}\nparams:#{params}")
+        SplitIoClient.configuration.logger.warn("#{e}\nURL:#{url}\ndata:#{data}\nparams:#{params}")
 
         false
       end
@@ -51,18 +51,18 @@ module SplitIoClient
         end
       end
 
-      def common_headers(api_key, config)
+      def common_headers(api_key)
         {
           'Authorization' => "Bearer #{api_key}",
-          'SplitSDKVersion' => "#{config.language}-#{config.version}",
-          'SplitSDKMachineName' => config.machine_name,
-          'SplitSDKMachineIP' => config.machine_ip,
-          'Referer' => referer(config)
+          'SplitSDKVersion' => "#{SplitIoClient.configuration.language}-#{SplitIoClient.configuration.version}",
+          'SplitSDKMachineName' => SplitIoClient.configuration.machine_name,
+          'SplitSDKMachineIP' => SplitIoClient.configuration.machine_ip,
+          'Referer' => referer
         }
       end
 
-      def referer(config)
-        result = "#{config.language}-#{config.version}"
+      def referer
+        result = "#{SplitIoClient.configuration.language}-#{SplitIoClient.configuration.version}"
 
         result = "#{result}::#{SplitIoClient::SplitConfig.machine_hostname}" unless SplitIoClient::SplitConfig.machine_hostname == 'localhost'
 

data/lib/splitclient-rb/engine/api/events.rb

@@ -1,22 +1,20 @@
 module SplitIoClient
   module Api
     class Events < Client
-      def initialize(api_key, config, events)
-        @config = config
+      def initialize(api_key, events)
         @api_key = api_key
         @events = events
       end
 
       def post
         if @events.empty?
-          @config.logger.debug('No events to report') if @config.debug_enabled
+          SplitIoClient.configuration.logger.debug('No events to report') if SplitIoClient.configuration.debug_enabled
           return
         end
 
-        @events.each_slice(@config.events_queue_size) do |event_slice|
+        @events.each_slice(SplitIoClient.configuration.events_queue_size) do |event_slice|
           result = post_api(
-            "#{@config.events_uri}/events/bulk",
-            @config,
+            "#{SplitIoClient.configuration.events_uri}/events/bulk",
             @api_key,
             event_slice.map { |event| formatted_event(event[:e]) },
             'SplitSDKMachineIP' => event_slice[0][:m][:i],
@@ -25,9 +23,9 @@ module SplitIoClient
           )
 
           if (200..299).include? result.status
-            @config.logger.debug("Events reported: #{event_slice.size}") if @config.debug_enabled
+            SplitIoClient.configuration.logger.debug("Events reported: #{event_slice.size}") if SplitIoClient.configuration.debug_enabled
           else
-            @config.logger.error("Unexpected status code while posting events: #{result.status}")
+            SplitIoClient.configuration.logger.error("Unexpected status code while posting events: #{result.status}")
           end
         end
       end