launchdarkly-server-sdk 8.18.0 → 8.18.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 04cc3611f092210f59ccf789aea80c573089e6c19884dd878a15695bb8affb24
-  data.tar.gz: 634641b8caf1b66a28e11e8e4d7321e0ab3e6fb4b6d8608f69a573c8e1ce56f4
+  metadata.gz: 911f611485b4aee247de0914c23169d847c130f4053669c363d55080d4c71ace
+  data.tar.gz: fed937b0ec2a0b265dc01049cff2c8453970f6480dbd21711dffece525d55ef1
 SHA512:
-  metadata.gz: 4602b9a3639878f3120e968fee5473f4f4abbdec95a0e4dd25e4bcfada163f04da868d8a258f51febc9890c36665520ac471523030e7015e065daef9d9e0180f
-  data.tar.gz: c536b25da3dc94de793d0114a07274ca00851caa2f73c4057ac707456bb77378ab2c5b21e9ccc805e68f1ee61ff1f52c6e4849c649139cab2e16a5bd3b1a13dd
+  metadata.gz: b77e71d79a713370c10733fee8b3d16808cc06635fdb6f63861e020687fe096f1f2befaec578d68d63878e1126c39098aa5b88abee3d92dab45ac1eb91cae0dd
+  data.tar.gz: f0f69217e0707543248265cc87f7186a97d0eadf1978420cf71b0b86df66f7960db765ba6414af5c5f9a62187e5de177bb5da94fe0a7f36b3c09dcd016d416a0

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

@@ -21,6 +21,7 @@ module LaunchDarkly
         @status_provider = BigSegmentStoreStatusProviderImpl.new(-> { get_status })
         @logger = logger
         @last_status = nil
+        @poll_lock = Mutex.new
 
         unless @store.nil?
           @cache = ExpiringCache.new(big_segments_config.context_cache_size, big_segments_config.context_cache_time)
@@ -49,18 +50,41 @@ module LaunchDarkly
             return BigSegmentMembershipResult.new(nil, BigSegmentsStatus::STORE_ERROR)
           end
         end
-        poll_store_and_update_status unless @last_status
-        unless @last_status.available
+        status = get_status
+        unless status.available
           return BigSegmentMembershipResult.new(membership, BigSegmentsStatus::STORE_ERROR)
         end
-        BigSegmentMembershipResult.new(membership, @last_status.stale ? BigSegmentsStatus::STALE : BigSegmentsStatus::HEALTHY)
+        BigSegmentMembershipResult.new(membership, status.stale ? BigSegmentsStatus::STALE : BigSegmentsStatus::HEALTHY)
       end
 
       def get_status
-        @last_status || poll_store_and_update_status
+        status = @last_status
+        return status if status
+
+        new_status = @poll_lock.synchronize do
+          # Another caller may have finished a poll while we waited for the lock.
+          status = @last_status
+          return status if status
+
+          query_store_status
+        end
+        @status_provider.update_status(new_status)
+
+        new_status
       end
 
       def poll_store_and_update_status
+        new_status = @poll_lock.synchronize { query_store_status }
+        @status_provider.update_status(new_status)
+
+        new_status
+      end
+
+      #
+      # Queries the store and caches the result. Callers hold @poll_lock, so this must not notify
+      # observers - a listener that calls back into the manager would deadlock on the mutex.
+      #
+      private def query_store_status
         new_status = Interfaces::BigSegmentStoreStatus.new(false, false) # default to "unavailable" if we don't get a new status below
         unless @store.nil?
           begin
@@ -71,9 +95,6 @@ module LaunchDarkly
           end
         end
         @last_status = new_status
-        @status_provider.update_status(new_status)
-
-        new_status
       end
 
       def stale?(timestamp)

data/lib/ldclient-rb/impl/data_source/polling.rb

@@ -38,14 +38,17 @@ module LaunchDarkly
           begin
             all_data, headers = request_all_data
             DataSource.record_environment_id(@config.data_source_update_sink, headers)
+            newly_initialized = false
             if all_data
               update_sink_or_data_store.init(all_data)
-              if @initialized.make_true
-                @config.logger.info { "[LDClient] Polling connection initialized" }
-                @ready.set
-              end
+              newly_initialized = @initialized.make_true
             end
             @config.data_source_update_sink&.update_status(LaunchDarkly::Interfaces::DataSource::Status::VALID, nil)
+
+            if newly_initialized
+              @config.logger.info { "[LDClient] Polling connection initialized" }
+              @ready.set
+            end
           rescue JSON::ParserError => e
             @config.logger.error { "[LDClient] JSON parsing failed for polling response." }
             error_info = LaunchDarkly::Interfaces::DataSource::ErrorInfo.new(

data/lib/ldclient-rb/impl/data_source/stream.rb

@@ -47,6 +47,7 @@ module LaunchDarkly
           opts = {
             headers: headers,
             read_timeout: READ_TIMEOUT_SECONDS,
+            connect_timeout: @config.connect_timeout,
             logger: @config.logger,
             socket_factory: @config.socket_factory,
             reconnect_time: @config.initial_reconnect_delay,

data/lib/ldclient-rb/impl/data_system/fdv2.rb

@@ -292,6 +292,10 @@ module LaunchDarkly
 
                 # Set ready event if and only if a selector is defined for the changeset.
                 if basis.change_set.selector && basis.change_set.selector.defined?
+                  @data_source_status_provider.update_status(
+                    LaunchDarkly::Interfaces::DataSource::Status::VALID,
+                    nil
+                  )
                   @ready_event.set
                   return fallback
                 end
@@ -473,15 +477,14 @@ module LaunchDarkly
               # Handle the update
               @store.apply(update.change_set, true) if update.change_set
 
-              # Set ready event on valid update
-              if update.state == LaunchDarkly::Interfaces::DataSource::Status::VALID
-                @ready_event.set
-                record_environment_id(update.environment_id)
-              end
+              valid = update.state == LaunchDarkly::Interfaces::DataSource::Status::VALID
+              record_environment_id(update.environment_id) if valid
 
               # Update status
               @data_source_status_provider.update_status(update.state, update.error)
 
+              @ready_event.set if valid
+
               return SyncResult::FDV1 if update.fallback_to_fdv1
 
               return SyncResult::REMOVE if update.state == LaunchDarkly::Interfaces::DataSource::Status::OFF

data/lib/ldclient-rb/impl/data_system/streaming.rb

@@ -97,6 +97,7 @@ module LaunchDarkly
           opts = {
             headers: headers,
             read_timeout: STREAM_READ_TIMEOUT,
+            connect_timeout: @http_config.connect_timeout,
             logger: @logger,
             socket_factory: @http_config.socket_factory,
             reconnect_time: @initial_reconnect_delay,

data/lib/ldclient-rb/impl/integrations/consul_impl.rb

@@ -68,13 +68,15 @@ module LaunchDarkly
 
           def get_all_internal(kind)
             items_out = {}
-            results = Diplomat::Kv.get(kind_key(kind), { recurse: true }, :return)
-            (results == "" ? [] : results).each do |result|
+            prefix = kind_key(kind)
+            Diplomat::Kv.get_all(prefix, {}, :return).each do |result|
               value = result[:value]
-              unless value.nil?
-                item = Model.deserialize(kind, value)
-                items_out[item[:key].to_sym] = item
-              end
+              next if value.nil?
+              db_key = result[:key].to_s
+              next unless db_key.start_with?(prefix)
+              # Use the key that the item is stored under, not the key inside the item. A deleted
+              # item (a "tombstone") is not guaranteed to carry a key of its own.
+              items_out[db_key[prefix.length..].to_sym] = Model.deserialize(kind, value)
             end
             items_out
           end

data/lib/ldclient-rb/impl/integrations/dynamodb_impl.rb

@@ -113,7 +113,10 @@ module LaunchDarkly
               resp = @client.query(req)
               resp.items.each do |item|
                 item_out = unmarshal_item(kind, item)
-                items_out[item_out[:key].to_sym] = item_out
+                next if item_out.nil?
+                # Use the sort key that the item is stored under, not the key inside the item. A
+                # deleted item (a "tombstone") is not guaranteed to carry a key of its own.
+                items_out[item[SORT_KEY].to_sym] = item_out
               end
               break if resp.last_evaluated_key.nil? || resp.last_evaluated_key.length == 0
               req.exclusive_start_key = resp.last_evaluated_key

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

@@ -23,6 +23,7 @@ module LaunchDarkly
 
         @mutex = Mutex.new # Covers the following variables
         @last_available = true
+        @stopped = false
         # @type [LaunchDarkly::Impl::RepeatingTask, nil]
         @poller = nil
       end
@@ -53,12 +54,14 @@ module LaunchDarkly
 
       def stop
         @store.stop
-        @mutex.synchronize do
-          return if @poller.nil?
 
-          @poller.stop
+        poller = @mutex.synchronize do
+          @stopped = true
+          task = @poller
           @poller = nil
+          task
         end
+        poller&.stop
       end
 
       def monitoring_enabled?
@@ -87,12 +90,12 @@ module LaunchDarkly
         @store_update_sink.update_status(status)
 
         if available
-          @mutex.synchronize do
-            return if @poller.nil?
-
-            @poller.stop
+          poller = @mutex.synchronize do
+            task = @poller
             @poller = nil
+            task
           end
+          poller&.stop
 
           return
         end
@@ -102,6 +105,9 @@ module LaunchDarkly
         task = Impl::RepeatingTask.new(0.5, 0, -> { self.check_availability }, @logger, 'LD/StoreWrapper#check_availability')
 
         @mutex.synchronize do
+          # A read can fail after stop, and a poller started then would never be stopped.
+          next if @stopped
+
           @poller = task
           @poller.start
         end

data/lib/ldclient-rb/version.rb

@@ -1,3 +1,3 @@
 module LaunchDarkly
-  VERSION = "8.18.0" # x-release-please-version
+  VERSION = "8.18.1" # x-release-please-version
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: launchdarkly-server-sdk
 version: !ruby/object:Gem::Version
-  version: 8.18.0
+  version: 8.18.1
 platform: ruby
 authors:
 - LaunchDarkly
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-09-16 00:00:00.000000000 Z
+date: 2026-10-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: aws-sdk-dynamodb