splitclient-rb 5.1.2.pre.rc21-java → 5.1.3.pre.rc1-java

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 1c06becb5108428c00b3ada0dd4404a22dd01c95
-  data.tar.gz: 214b5ade2b8de67da2990b2370bc05936f7a7d87
+  metadata.gz: 86614a077b8bb2e11605c201e8d7ce74fa53af08
+  data.tar.gz: 5a775b16d37564ba6897dccd48d33472e7c4e1c3
 SHA512:
-  metadata.gz: 0376aa2502f50ec6f986d67112ce44cb0c4be8484648d41b02c10ca3fda71dae087e0a43a960b261d922c0a4fb706d320dd655cba3a592cca56667f8c7a94a57
-  data.tar.gz: b73de50c0959735428176349352694d2cc778390732a9bb6885a5089a35ad7e95171ffd7a58a7cad4a6f39858e9124b01979036d28f614e413371f87cc290f66
+  metadata.gz: 22d0552dfae11161414e2a624ffaf8a4e7d402914541109da17f71178474b6627e6b5c5ef71622e56190cbb1b30cec8cba60c2bf39099b0019f8705ed7c4f71a
+  data.tar.gz: 90e26ace99de500d55c05ddd9dc632b17e9d67bc8a1b5abbd1facae06110dfb8a43cb5443b9172293da7a25e1156a409b8e05406c7be5c1475ef307e33d70011

data/.gitignore

@@ -44,5 +44,8 @@ lib/murmurhash/murmurhash.so
 ext/murmurhash/murmurhash.bundle
 ext/murmurhash/murmurhash.so
 
-#Ignore dump.rdb
+# Ignore dump.rdb
 dump.rdb
+
+# Ignore Mac OS generated files
+.DS_Store

data/CHANGES.txt

@@ -1,3 +1,9 @@
+5.1.3
+- Add cache wrapper to treatments and segments.
+
+5.1.2 (October 26th, 2018)
+- Add input validation for client API methods
+
 5.1.1 (October 4th, 2018)
 - Change get_treatments so that it sends a single latency metric
 - Removed unused call to Redis#scan when adding latencies

data/Detailed-README.md

@@ -301,6 +301,14 @@ _To use Redis, include `redis-rb` in your app's Gemfile._
 
 *default value* = (your current hostname)
 
+**cache_ttl** : Time to live in seconds for the memory cache values when using Redis.
+
+*default value* = `5`
+
+**max_cache_size** : Maximum number of items held in the memory cache values when using Redis. When cache is full an LRU strategy for pruning shall be used.
+
+*default value* = `500`
+
 **redis_url** : Redis URL or hash with configuration for the SDK to connect to. See [Redis#initialize](https://www.rubydoc.info/github/redis/redis-rb/Redis%3Ainitialize) for detailed information.
 
 *default value* = `'redis://127.0.0.1:6379/0'`

data/NEWS

@@ -1,3 +1,11 @@
+5.1.3
+
+Add cache wrapper to treatments and segments.
+
+5.1.2
+
+Add input validation for client API methods: get_treatment, get_treatments, track, manager
+
 5.1.1
 
 Reduces the number of calls to Redis when calling #client.get_treatments using such cache adapter.

data/lib/splitclient-rb.rb

@@ -2,11 +2,11 @@ require 'forwardable'
 
 require 'splitclient-rb/version'
 
-require 'splitclient-rb/exceptions/impressions_shutdown_exception'
-require 'splitclient-rb/exceptions/sdk_blocker_timeout_expired_exception'
+require 'splitclient-rb/exceptions'
 require 'splitclient-rb/cache/routers/impression_router'
 require 'splitclient-rb/cache/adapters/memory_adapters/map_adapter'
 require 'splitclient-rb/cache/adapters/memory_adapters/queue_adapter'
+require 'splitclient-rb/cache/adapters/cache_adapter'
 require 'splitclient-rb/cache/adapters/memory_adapter'
 require 'splitclient-rb/cache/adapters/redis_adapter'
 require 'splitclient-rb/cache/repositories/repository'
@@ -80,6 +80,7 @@ require 'splitclient-rb/engine/models/split'
 require 'splitclient-rb/engine/models/label'
 require 'splitclient-rb/engine/models/treatment'
 require 'splitclient-rb/utilitites'
+require 'splitclient-rb/validators'
 
 # C extension
 require 'murmurhash/murmurhash_mri'

data/lib/splitclient-rb/cache/adapters/cache_adapter.rb

@@ -0,0 +1,131 @@
+require 'lru_redux'
+
+module SplitIoClient
+  module Cache
+    module Adapters
+      class CacheAdapter
+        extend Forwardable
+        def_delegators :@adapter, :initialize_set, :set_bool, :pipelined
+
+        def initialize(adapter)
+          @cache = LruRedux::TTL::ThreadSafeCache.new(SplitIoClient.configuration.max_cache_size, SplitIoClient.configuration.cache_ttl)
+          @adapter = adapter
+        end
+
+        def delete(key)
+          @cache.delete(key)
+          @adapter.delete(key)
+        end
+
+        def clear(namespace_key)
+          @cache.clear
+          @adapter.clear(namespace_key)
+        end
+
+        def string(key)
+          value = get(key)
+          return value if value
+          value = @adapter.string(key)
+          add(key, value)
+          value
+        end
+
+        def set_string(key, value)
+          add(key, value)
+          @adapter.set_string(key, value)
+        end
+
+        def multiple_strings(keys)
+          cached_values = keys.each_with_object({}) do |key, memo|
+             memo[key] = get(key)
+          end
+
+          non_cached_keys = []
+          cached_values.delete_if{ |k,v| v.nil? ? non_cached_keys << k : false }
+
+          if non_cached_keys.any?
+            new_values = @adapter.multiple_strings(non_cached_keys)
+
+            new_values.keys.each do |key, value|
+              add(key, value)
+            end
+
+            cached_values.merge!(new_values)
+          end
+
+          cached_values
+        end
+
+        def find_strings_by_prefix(prefix)
+          @adapter.find_strings_by_prefix(prefix)
+        end
+
+        def exists?(key)
+          cached_value = get(key)
+          if cached_value.nil?
+            @adapter.exists?(key)
+          else
+            true
+          end
+        end
+
+        def add_to_set(key, values)
+          if values.is_a? Array
+            values.each { |value| add_to_map(key, value, 1) }
+          else
+            add_to_map(key, values, 1)
+          end
+          @adapter.add_to_set(key, values)
+        end
+
+        def in_set?(key, field)
+          cached_value = get(key)
+          if cached_value.nil?
+            return @adapter.in_set?(key, field)
+          end
+          cached_value.key?(field)
+        end
+
+        def get_set(key)
+          cached_value = get(key)
+          if cached_value.nil?
+            return @adapter.get_set(key)
+          end
+          cached_value.keys
+        end
+
+        def delete_from_set(key, fields)
+          cached_value = get(key)
+          if cached_value
+            if fields.is_a? Array
+              fields.each { |field| cached_value.delete(field) }
+            else
+              cached_value.delete(fields)
+            end
+          end
+
+          @adapter.delete_from_set(key, fields)
+        end
+
+        def initialize_map(key)
+          @cache[key] = Concurrent::Map.new
+        end
+
+        private
+
+        def add_to_map(key, field, value)
+          initialize_map(key) unless get(key)
+          get(key).put(field.to_s, value.to_s)
+        end
+
+        def add(key, value)
+          @cache[key] = value.to_s unless value.nil?
+        end
+
+        def get(key)
+          @cache[key]
+        end
+      end
+    end
+  end
+end

data/lib/splitclient-rb/cache/repositories/segments_repository.rb

@@ -7,7 +7,12 @@ module SplitIoClient
         attr_reader :adapter
 
         def initialize(adapter)
-          @adapter = adapter
+          @adapter = case adapter.class.to_s
+          when 'SplitIoClient::Cache::Adapters::RedisAdapter'
+            SplitIoClient::Cache::Adapters::CacheAdapter.new(adapter)
+          else
+            adapter
+          end
           @adapter.set_bool(namespace_key('.ready'), false) unless SplitIoClient.configuration.mode == :consumer
         end
 

data/lib/splitclient-rb/cache/repositories/splits_repository.rb

@@ -7,7 +7,12 @@ module SplitIoClient
         attr_reader :adapter
 
         def initialize(adapter)
-          @adapter = adapter
+          @adapter = case adapter.class.to_s
+          when 'SplitIoClient::Cache::Adapters::RedisAdapter'
+            SplitIoClient::Cache::Adapters::CacheAdapter.new(adapter)
+          else
+            adapter
+          end
           unless SplitIoClient.configuration.mode == :consumer
             @adapter.set_string(namespace_key('.splits.till'), '-1')
             @adapter.initialize_map(namespace_key('.segments.registered'))
@@ -26,7 +31,7 @@ module SplitIoClient
 
         def get_splits(names)
           splits = {}
-          split_names = names.reject(&:empty?).uniq.map { |name| namespace_key(".split.#{name}") }
+          split_names = names.map { |name| namespace_key(".split.#{name}") }
           splits.merge!(
             @adapter
               .multiple_strings(split_names)

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

@@ -1,4 +1,5 @@
 module SplitIoClient
+
   class SplitClient
     #
     # Creates a new split client instance that connects to split.io API.
@@ -17,11 +18,24 @@ module SplitIoClient
     end
 
     def get_treatments(key, split_names, attributes = {})
+
+      return nil unless SplitIoClient::Validators.valid_get_treatments_parameters(split_names)
+
+      sanitized_split_names = sanitize_split_names(split_names)
+
+      if sanitized_split_names.empty?
+        SplitIoClient.configuration.logger.warn('get_treatments: split_names is an empty array or has null values')
+        return {}
+      end
+
       bucketing_key, matching_key = keys_from_key(key)
+      bucketing_key = bucketing_key ? bucketing_key.to_s : nil
+      matching_key = matching_key ? matching_key.to_s : nil
+
       evaluator = Engine::Parser::Evaluator.new(@segments_repository, @splits_repository, true)
       start = Time.now
       treatments_labels_change_numbers =
-        @splits_repository.get_splits(split_names).each_with_object({}) do |(name, data), memo|
+        @splits_repository.get_splits(sanitized_split_names).each_with_object({}) do |(name, data), memo|
           memo.merge!(name => get_treatment(key, name, attributes, data, false, true, evaluator))
         end
       latency = (Time.now - start) * 1000.0
@@ -34,13 +48,13 @@ module SplitIoClient
           matching_key, bucketing_key, treatments_labels_change_numbers, time
         )
 
-        route_impressions(split_names, matching_key, bucketing_key, time, treatments_labels_change_numbers, attributes)
+        route_impressions(sanitized_split_names, matching_key, bucketing_key, time, treatments_labels_change_numbers, attributes)
       end
 
-      split_names = treatments_labels_change_numbers.keys
+      split_names_keys = treatments_labels_change_numbers.keys
       treatments = treatments_labels_change_numbers.values.map { |v| v[:treatment] }
 
-      Hash[split_names.zip(treatments)]
+      Hash[split_names_keys.zip(treatments)]
     end
 
     #
@@ -59,68 +73,43 @@ module SplitIoClient
         key, split_name, attributes = {}, split_data = nil, store_impressions = true,
         multiple = false, evaluator = nil
       )
-      bucketing_key, matching_key = keys_from_key(key)
-      treatment_data = { label: Engine::Models::Label::DEFINITION_NOT_FOUND, treatment: SplitIoClient::Engine::Models::Treatment::CONTROL }
-      evaluator ||= Engine::Parser::Evaluator.new(@segments_repository, @splits_repository)
+      control_treatment = { label: Engine::Models::Label::EXCEPTION, treatment: SplitIoClient::Engine::Models::Treatment::CONTROL }
+      parsed_control_treatment = parsed_treatment(multiple, control_treatment)
 
-      if matching_key.nil?
-        SplitIoClient.configuration.logger.warn('matching_key was null for split_name: ' + split_name.to_s)
-        return parsed_treatment(multiple, treatment_data)
-      end
+      bucketing_key, matching_key = keys_from_key(key)
 
-      if split_name.nil?
-        SplitIoClient.configuration.logger.warn('split_name was null for key: ' + key)
-        return parsed_treatment(multiple, treatment_data)
-      end
+      return parsed_control_treatment unless SplitIoClient::Validators.valid_get_treatment_parameters(key, split_name, matching_key, bucketing_key)
 
-      start = Time.now
+      bucketing_key = bucketing_key ? bucketing_key.to_s : nil
+      matching_key = matching_key.to_s
+      evaluator ||= Engine::Parser::Evaluator.new(@segments_repository, @splits_repository)
 
       begin
+        start = Time.now
+
         split = multiple ? split_data : @splits_repository.get_split(split_name)
 
         if split.nil?
-          SplitIoClient.configuration.logger.debug("split_name: #{split_name} does not exist. Returning CONTROL")
-          return parsed_treatment(multiple, treatment_data)
-        else
-          treatment_data =
-            evaluator.call(
-            { bucketing_key: bucketing_key, matching_key: matching_key }, split, attributes
-          )
+          SplitIoClient.configuration.logger.warn("split_name: #{split_name} does not exist. Returning CONTROL")
+          return parsed_control_treatment
         end
-      rescue StandardError => error
-        SplitIoClient.configuration.log_found_exception(__method__.to_s, error)
 
-        store_impression(
-          split_name, matching_key, bucketing_key,
-          {
-            treatment: SplitIoClient::Engine::Models::Treatment::CONTROL,
-            label: SplitIoClient::Engine::Models::Label::EXCEPTION
-          },
-          store_impressions, attributes
+        treatment_data =
+          evaluator.call(
+          { bucketing_key: bucketing_key, matching_key: matching_key }, split, attributes
         )
 
-        return parsed_treatment(multiple, treatment_data)
-      end
-
-      begin
         latency = (Time.now - start) * 1000.0
-        split && store_impression(split_name, matching_key, bucketing_key, treatment_data, store_impressions, attributes)
+        store_impression(split_name, matching_key, bucketing_key, treatment_data, store_impressions, attributes)
 
         # Measure
         @adapter.metrics.time('sdk.get_treatment', latency) unless multiple
       rescue StandardError => error
         SplitIoClient.configuration.log_found_exception(__method__.to_s, error)
 
-        store_impression(
-          split_name, matching_key, bucketing_key,
-          {
-            treatment: SplitIoClient::Engine::Models::Treatment::CONTROL,
-            label: SplitIoClient::Engine::Models::Label::EXCEPTION
-          },
-          store_impressions, attributes
-        )
+        store_impression(split_name, matching_key, bucketing_key, control_treatment, store_impressions, attributes)
 
-        return parsed_treatment(multiple, treatment_data)
+        return parsed_control_treatment
       end
 
       parsed_treatment(multiple, treatment_data)
@@ -188,16 +177,23 @@ module SplitIoClient
       @impression_router ||= SplitIoClient::ImpressionRouter.new
     end
 
-    def track(key, traffic_type, event_type, value = nil)
-      @events_repository.add(key, traffic_type, event_type, (Time.now.to_f * 1000).to_i, value)
+    def track(key, traffic_type_name, event_type, value = nil)
+      return false unless SplitIoClient::Validators.valid_track_parameters(key, traffic_type_name, event_type, value)
+      begin
+        @events_repository.add(key.to_s, traffic_type_name, event_type.to_s, (Time.now.to_f * 1000).to_i, value)
+        true
+      rescue StandardError => error
+        SplitIoClient.configuration.log_found_exception(__method__.to_s, error)
+        false
+      end
     end
 
     def keys_from_key(key)
       case key.class.to_s
       when 'Hash'
-        key.values_at(:bucketing_key, :matching_key).map { |k| k.nil? ? nil : k.to_s }
+        key.values_at(:bucketing_key, :matching_key).map { |k| k.nil? ? nil : k }
       else
-        [nil, key].map { |k| k.nil? ? nil : k.to_s }
+        [nil, key].map { |k| k.nil? ? nil : k }
       end
     end
 
@@ -212,5 +208,16 @@ module SplitIoClient
         treatment_data[:treatment]
       end
     end
+
+    def sanitize_split_names(split_names)
+      split_names.compact.uniq.select do |split_name|
+        if split_name.is_a?(String) && !split_name.empty?
+          true
+        else
+          SplitIoClient.configuration.logger.warn('get_treatments: split_name has to be a non empty string')
+          false
+        end
+      end
+    end
   end
 end

data/lib/splitclient-rb/engine/models/label.rb

@@ -4,5 +4,4 @@ class SplitIoClient::Engine::Models::Label
   EXCEPTION = 'exception'.freeze
   KILLED = 'killed'.freeze
   NOT_IN_SPLIT = 'not in split'.freeze
-  DEFINITION_NOT_FOUND = 'definition not found'.freeze
 end

data/lib/splitclient-rb/exceptions.rb

@@ -0,0 +1,7 @@
+module SplitIoClient
+  class SplitIoError < StandardError; end
+
+  class ImpressionShutdownException < SplitIoError; end
+
+  class SDKBlockerTimeoutExpiredException < SplitIoError; end
+end

data/lib/splitclient-rb/managers/split_manager.rb

@@ -43,7 +43,7 @@ module SplitIoClient
     #
     # @returns a split view
     def split(split_name)
-      return unless @splits_repository
+      return unless @splits_repository && SplitIoClient::Validators.valid_split_parameters(split_name)
 
       split = @splits_repository.get_split(split_name)
 
@@ -63,7 +63,6 @@ module SplitIoClient
         treatments = []
       end
 
-
         {
           name: name,
           traffic_type_name: split[:trafficTypeName],

data/lib/splitclient-rb/split_config.rb

@@ -33,6 +33,8 @@ module SplitIoClient
     # @option opts [Int] :impressions_queue_size Size of the impressions queue in the memory repository. Once reached, newer impressions will be dropped
     # @option opts [Int] :impressions_bulk_size Max number of impressions to be sent to the backend on each post
     # @option opts [#log] :impression_listener this object will capture all impressions and process them through `#log`
+    # @option opts [Int] :cache_ttl Time to live in seconds for the memory cache values when using Redis.
+    # @option opts [Int] :max_cache_size Max number of items to be held in the memory cache before prunning when using Redis.
     # @return [type] SplitConfig with configuration options
     def initialize(opts = {})
       @base_uri = (opts[:base_uri] || SplitConfig.default_base_uri).chomp('/')
@@ -70,6 +72,9 @@ module SplitIoClient
       @machine_name = opts[:machine_name] || SplitConfig.machine_hostname
       @machine_ip = opts[:machine_ip] || SplitConfig.machine_ip
 
+      @cache_ttl = opts[:cache_ttl] || SplitConfig.cache_ttl
+      @max_cache_size = opts[:max_cache_size] || SplitConfig.max_cache_size
+
       @language = opts[:language] || 'ruby'
       @version = opts[:version] || SplitIoClient::VERSION
 
@@ -176,6 +181,9 @@ module SplitIoClient
     attr_accessor :machine_ip
     attr_accessor :machine_name
 
+    attr_accessor :cache_ttl
+    attr_accessor :max_cache_size
+
     attr_accessor :language
     attr_accessor :version
 
@@ -357,6 +365,21 @@ module SplitIoClient
       false
     end
 
+    #
+    # The default cache time to live
+    #
+    # @return [boolean]
+    def self.cache_ttl
+      5
+    end
+
+    # The default max cache size
+    #
+    # @return [boolean]
+    def self.max_cache_size
+      500
+    end
+
     #
     # custom logger of exceptions
     #

data/lib/splitclient-rb/validators.rb

@@ -0,0 +1,185 @@
+module SplitIoClient
+  module Validators
+    extend self
+
+    def valid_get_treatment_parameters(key, split_name, matching_key, bucketing_key)
+      valid_key?(key) && valid_split_name?(split_name) && valid_matching_key?(matching_key) && valid_bucketing_key?(bucketing_key)
+    end
+
+    def valid_get_treatments_parameters(split_names)
+      valid_split_names?(split_names)
+    end
+
+    def valid_track_parameters(key, traffic_type_name, event_type, value)
+      valid_track_key?(key) && valid_traffic_type_name?(traffic_type_name) && valid_event_type?(event_type) && valid_value?(value)
+    end
+
+    def valid_split_parameters(split_name)
+      valid_split_name?(split_name, :split)
+    end
+
+    private
+
+    def string?(value)
+      value.is_a?(String) || value.is_a?(Symbol)
+    end
+
+    def number_or_string?(value)
+      value.is_a?(Numeric) || string?(value)
+    end
+
+    def log_nil(key, method)
+      SplitIoClient.configuration.logger.error("#{method}: #{key} cannot be nil")
+    end
+
+    def log_string(key, method)
+      SplitIoClient.configuration.logger.error("#{method}: #{key} must be a String or a Symbol")
+    end
+
+    def log_number_or_string(key, method)
+      SplitIoClient.configuration.logger.error("#{method}: #{key} must be a String")
+    end
+
+    def log_convert_numeric(key, method)
+      SplitIoClient.configuration.logger.warn("#{method}: #{key} is not of type String, converting to String")
+    end
+
+    def valid_split_name?(split_name, method=:get_treatment)
+      if split_name.nil?
+        log_nil(:split_name, method)
+        return false
+      end
+
+      unless string?(split_name)
+        log_string(:split_name, method)
+        return false
+      end
+
+      return true
+    end
+
+    def valid_key?(key)
+      if key.nil?
+        log_nil(:key, :get_treatment)
+        return false
+      end
+
+      return true
+    end
+
+    def valid_matching_key?(matching_key)
+      if matching_key.nil?
+        log_nil(:matching_key, :get_treatment)
+        return false
+      end
+
+      unless number_or_string?(matching_key)
+        log_number_or_string(:matching_key, :get_treatment)
+        return false
+      end
+
+      if matching_key.is_a? Numeric
+        log_convert_numeric(:matching_key, :get_treatment)
+      end
+
+      return true
+    end
+
+    def valid_bucketing_key?(bucketing_key)
+      if bucketing_key.nil?
+        SplitIoClient.configuration.logger.warn('get_treatment: key object should have bucketing_key set')
+        return true
+      end
+
+      unless number_or_string?(bucketing_key)
+        log_number_or_string(:bucketing_key, :get_treatment)
+        return false
+      end
+
+      if bucketing_key.is_a? Numeric
+        log_convert_numeric(:bucketing_key, :get_treatment)
+      end
+
+      return true
+    end
+
+    def valid_split_names?(split_names)
+      if split_names.nil?
+        log_nil(:split_names, :get_treatments)
+        return false
+      end
+
+      unless split_names.is_a? Array
+        SplitIoClient.configuration.logger.warn('get_treatments: split_names must be an Array')
+        return false
+      end
+
+      return true
+    end
+
+    def valid_track_key?(key)
+      if key.nil?
+        log_nil(:key, :track)
+        return false
+      end
+
+      unless number_or_string?(key)
+        log_number_or_string(:key, :track)
+        return false
+      end
+
+      if key.is_a? Numeric
+        log_convert_numeric(:key, :track)
+      end
+
+      return true
+    end
+
+    def valid_event_type?(event_type)
+      if event_type.nil?
+        log_nil(:event_type, :track)
+        return false
+      end
+
+      unless string?(event_type)
+        log_string(:event_type, :track)
+        return false
+      end
+
+      if (event_type.to_s =~ /[a-zA-Z0-9][-_\.a-zA-Z0-9]{0,62}/).nil?
+        SplitIoClient.configuration.logger.error('track: event_type must adhere to [a-zA-Z0-9][-_\.a-zA-Z0-9]{0,62}')
+        return false
+      end
+
+      return true
+    end
+
+    def valid_traffic_type_name?(traffic_type_name)
+      if traffic_type_name.nil?
+        log_nil(:traffic_type_name, :track)
+        return false
+      end
+
+      unless string?(traffic_type_name)
+        log_string(:traffic_type_name, :track)
+        return false
+      end
+
+      if traffic_type_name.empty?
+        SplitIoClient.configuration.logger.error('track: traffic_type_name must not be an empty String')
+        return false
+      end
+
+      return true
+    end
+
+    def valid_value?(value)
+      unless value.is_a?(Numeric) || value.nil?
+        SplitIoClient.configuration.logger.error('track: value must be a number')
+        return false
+      end
+
+      return true
+    end
+  end
+end

data/lib/splitclient-rb/version.rb

@@ -1,3 +1,3 @@
 module SplitIoClient
-  VERSION = '5.1.2.pre.rc21'
+  VERSION = '5.1.3.pre.rc1'
 end

data/splitclient-rb.gemspec

@@ -46,11 +46,13 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency 'rspec'
   spec.add_development_dependency 'rubocop', '0.59.0'
   spec.add_development_dependency 'simplecov'
+  spec.add_development_dependency 'timecop'
   spec.add_development_dependency 'webmock'
 
   spec.add_runtime_dependency 'concurrent-ruby', '~> 1.0'
   spec.add_runtime_dependency 'faraday', '>= 0.8'
   spec.add_runtime_dependency 'json', '>= 1.8'
+  spec.add_runtime_dependency 'lru_redux'
   spec.add_runtime_dependency 'net-http-persistent', '~> 2.9'
   spec.add_runtime_dependency 'redis', '>= 3.2'
   spec.add_runtime_dependency 'thread_safe', '>= 0.3'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: splitclient-rb
 version: !ruby/object:Gem::Version
-  version: 5.1.2.pre.rc21
+  version: 5.1.3.pre.rc1
 platform: java
 authors:
 - Split Software
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2018-10-18 00:00:00.000000000 Z
+date: 2018-11-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
@@ -136,6 +136,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  name: timecop
+  prerelease: false
+  type: :development
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
     requirements:
@@ -192,6 +206,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '1.8'
+- !ruby/object:Gem::Dependency
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  name: lru_redux
+  prerelease: false
+  type: :runtime
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
     requirements:
@@ -257,6 +285,7 @@ files:
 - lib/murmurhash/murmurhash.jar
 - lib/murmurhash/murmurhash_mri.rb
 - lib/splitclient-rb.rb
+- lib/splitclient-rb/cache/adapters/cache_adapter.rb
 - lib/splitclient-rb/cache/adapters/memory_adapter.rb
 - lib/splitclient-rb/cache/adapters/memory_adapters/map_adapter.rb
 - lib/splitclient-rb/cache/adapters/memory_adapters/queue_adapter.rb
@@ -321,8 +350,7 @@ files:
 - lib/splitclient-rb/engine/parser/evaluator.rb
 - lib/splitclient-rb/engine/parser/partition.rb
 - lib/splitclient-rb/engine/parser/split_adapter.rb
-- lib/splitclient-rb/exceptions/impressions_shutdown_exception.rb
-- lib/splitclient-rb/exceptions/sdk_blocker_timeout_expired_exception.rb
+- lib/splitclient-rb/exceptions.rb
 - lib/splitclient-rb/localhost_split_factory.rb
 - lib/splitclient-rb/localhost_utils.rb
 - lib/splitclient-rb/managers/localhost_split_manager.rb
@@ -332,6 +360,7 @@ files:
 - lib/splitclient-rb/split_factory_builder.rb
 - lib/splitclient-rb/split_logger.rb
 - lib/splitclient-rb/utilitites.rb
+- lib/splitclient-rb/validators.rb
 - lib/splitclient-rb/version.rb
 - splitclient-rb.gemspec
 - splitio.yml.example

data/lib/splitclient-rb/exceptions/impressions_shutdown_exception.rb

@@ -1,4 +0,0 @@
-module SplitIoClient
-  class ImpressionShutdownException < StandardError
-  end
-end

data/lib/splitclient-rb/exceptions/sdk_blocker_timeout_expired_exception.rb

@@ -1,4 +0,0 @@
-module SplitIoClient
-  class SDKBlockerTimeoutExpiredException < StandardError
-  end
-end