launchdarkly-server-sdk 8.11.1 → 8.11.3

data/lib/ldclient-rb/impl/util.rb

@@ -1,3 +1,6 @@
+require "uri"
+require "http"
+
 module LaunchDarkly
   module Impl
     module Util
@@ -93,6 +96,68 @@ module LaunchDarkly
         }
         nil
       end
+
+      #
+      # Append the payload filter key query parameter to the provided URI.
+      #
+      # @param uri [String]
+      # @param config [Config]
+      # @return [String]
+      #
+      def self.add_payload_filter_key(uri, config)
+        return uri if config.payload_filter_key.nil?
+
+        begin
+          parsed = URI.parse(uri)
+          new_query_params = URI.decode_www_form(String(parsed.query)) << ["filter", config.payload_filter_key]
+          parsed.query = URI.encode_www_form(new_query_params)
+          parsed.to_s
+        rescue URI::InvalidURIError
+          config.logger.warn { "[LDClient] URI could not be parsed. No filtering will be applied." }
+          uri
+        end
+      end
+
+      def self.new_http_client(uri_s, config)
+        http_client_options = {}
+        if config.socket_factory
+          http_client_options["socket_class"] = config.socket_factory
+        end
+        proxy = URI.parse(uri_s).find_proxy
+        unless proxy.nil?
+          http_client_options["proxy"] = {
+            proxy_address: proxy.host,
+            proxy_port: proxy.port,
+            proxy_username: proxy.user,
+            proxy_password: proxy.password,
+          }
+        end
+        HTTP::Client.new(http_client_options)
+          .timeout({
+            read: config.read_timeout,
+            connect: config.connect_timeout,
+          })
+          .persistent(uri_s)
+      end
+
+      def self.log_exception(logger, message, exc)
+        logger.error { "[LDClient] #{message}: #{exc.inspect}" }
+        logger.debug { "[LDClient] Exception trace: #{exc.backtrace}" }
+      end
+
+      def self.http_error_recoverable?(status)
+        if status >= 400 && status < 500
+          status == 400 || status == 408 || status == 429
+        else
+          true
+        end
+      end
+
+      def self.http_error_message(status, context, recoverable_message)
+        desc = (status == 401 || status == 403) ? " (invalid SDK key)" : ""
+        message = http_error_recoverable?(status) ? recoverable_message : "giving up permanently"
+        "HTTP error #{status}#{desc} for #{context} - #{message}"
+      end
     end
   end
 end

data/lib/ldclient-rb/impl.rb

@@ -5,8 +5,7 @@ module LaunchDarkly
   # and subject to change.
   #
   # @since 5.5.0
-  # @private
-  #
+  # @api private
   module Impl
     # code is in ldclient-rb/impl/
   end

data/lib/ldclient-rb/in_memory_store.rb

@@ -1,24 +1,7 @@
 require "concurrent/atomics"
+require "ldclient-rb/impl/data_store"
 
 module LaunchDarkly
-
-  # These constants denote the types of data that can be stored in the feature store.  If
-  # we add another storable data type in the future, as long as it follows the same pattern
-  # (having "key", "version", and "deleted" properties), we only need to add a corresponding
-  # constant here and the existing store should be able to handle it.
-  #
-  # The :priority and :get_dependency_keys properties are used by FeatureStoreDataSetSorter
-  # to ensure data consistency during non-atomic updates.
-
-  # @private
-  FEATURES = Impl::DataStore::DataKind.new(namespace: "features", priority: 1).freeze
-
-  # @private
-  SEGMENTS = Impl::DataStore::DataKind.new(namespace: "segments", priority: 0).freeze
-
-  # @private
-  ALL_KINDS = [FEATURES, SEGMENTS].freeze
-
   #
   # Default implementation of the LaunchDarkly client's feature store, using an in-memory
   # cache.  This object holds feature flags and related data received from LaunchDarkly.

data/lib/ldclient-rb/integrations/test_data/flag_builder.rb

@@ -12,14 +12,14 @@ module LaunchDarkly
       class FlagBuilder
         attr_reader :key
 
-        # @private
+        # @api private
         def initialize(key)
           @key = key
           @on = true
           @variations = []
         end
 
-        # @private
+        # @api private
         def initialize_copy(other)
           super(other)
           @variations = @variations.clone
@@ -357,7 +357,7 @@ module LaunchDarkly
           self
         end
 
-        # @private
+        # @api private
         def add_rule(rule)
           if @rules.nil?
             @rules = Array.new
@@ -386,7 +386,7 @@ module LaunchDarkly
           end
         end
 
-        # @private
+        # @api private
         def build(version)
           res = { key: @key,
                   version: version,
@@ -486,16 +486,16 @@ module LaunchDarkly
         # Finally, call {#then_return} to finish defining the rule.
         #
         class FlagRuleBuilder
-          # @private
+          # @api private
           FlagRuleClause = Struct.new(:contextKind, :attribute, :op, :values, :negate, keyword_init: true) # rubocop:disable Naming/MethodName:
 
-          # @private
+          # @api private
           def initialize(flag_builder)
             @flag_builder = flag_builder
             @clauses = Array.new
           end
 
-          # @private
+          # @api private
           def intialize_copy(other)
             super(other)
             @clauses = @clauses.clone
@@ -612,7 +612,7 @@ module LaunchDarkly
             end
           end
 
-          # @private
+          # @api private
           def build(ri)
             {
               id: 'rule' + ri.to_s,
@@ -622,7 +622,7 @@ module LaunchDarkly
           end
         end
 
-        # @private
+        # @api private
         def variation_for_boolean(variation)
           variation ? TRUE_VARIATION_INDEX : FALSE_VARIATION_INDEX
         end

data/lib/ldclient-rb/integrations/test_data.rb

@@ -42,7 +42,7 @@ module LaunchDarkly
         self.new
       end
 
-      # @private
+      # @api private
       def initialize
         @flag_builders = Hash.new
         @current_flags = Hash.new
@@ -56,7 +56,7 @@ module LaunchDarkly
       # Called internally by the SDK to determine what arguments to pass to call
       # You do not need to call this method.
       #
-      # @private
+      # @api private
       def arity
         2
       end
@@ -65,7 +65,7 @@ module LaunchDarkly
       # Called internally by the SDK to associate this test data source with an {@code LDClient} instance.
       # You do not need to call this method.
       #
-      # @private
+      # @api private
       def call(_, config)
         impl = LaunchDarkly::Impl::Integrations::TestData::TestDataSource.new(config.feature_store, self)
         @instances_lock.with_write_lock { @instances.push(impl) }
@@ -121,10 +121,10 @@ module LaunchDarkly
           if @current_flags[flag_key]
             version = @current_flags[flag_key][:version]
           end
-          new_flag = Impl::Model.deserialize(FEATURES, flag_builder.build(version+1))
+          new_flag = LaunchDarkly::Impl::Model.deserialize(LaunchDarkly::Impl::DataStore::FEATURES, flag_builder.build(version+1))
           @current_flags[flag_key] = new_flag
         end
-        update_item(FEATURES, new_flag)
+        update_item(LaunchDarkly::Impl::DataStore::FEATURES, new_flag)
         self
       end
 
@@ -147,7 +147,7 @@ module LaunchDarkly
       # @return [TestData] the TestData instance
       #
       def use_preconfigured_flag(flag)
-        use_preconfigured_item(FEATURES, flag, @current_flags)
+        use_preconfigured_item(LaunchDarkly::Impl::DataStore::FEATURES, flag, @current_flags)
       end
 
       #
@@ -167,7 +167,7 @@ module LaunchDarkly
       # @return [TestData] the TestData instance
       #
       def use_preconfigured_segment(segment)
-        use_preconfigured_item(SEGMENTS, segment, @current_segments)
+        use_preconfigured_item(LaunchDarkly::Impl::DataStore::SEGMENTS, segment, @current_segments)
       end
 
       private def use_preconfigured_item(kind, item, current)
@@ -194,17 +194,17 @@ module LaunchDarkly
         end
       end
 
-      # @private
+      # @api private
       def make_init_data
         @lock.with_read_lock do
           {
-            FEATURES => @current_flags.clone,
-            SEGMENTS => @current_segments.clone,
+            LaunchDarkly::Impl::DataStore::FEATURES => @current_flags.clone,
+            LaunchDarkly::Impl::DataStore::SEGMENTS => @current_segments.clone,
           }
         end
       end
 
-      # @private
+      # @api private
       def closed_instance(instance)
         @instances_lock.with_write_lock { @instances.delete(instance) }
       end