splitclient-rb 7.3.3.pre.rc14 → 7.3.3.pre.rc18

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 791cffe64de83620476b94bf8c3845d20ff60c2f3d005ef8936a592c286be927
-  data.tar.gz: 1135f45f3fbc2105265ab9e0ba9ffaf80c447e1a941d6481813b1afa20c3063f
+  metadata.gz: 463a9d41ab4651544984dc087f5800d6e86566a694c8ca0929c224c145bed839
+  data.tar.gz: e8a870ca9198ff396ca9d5f621d86623acb08cc081ea145623c27f0b8571565d
 SHA512:
-  metadata.gz: a8622cd332b006082f590d8b2459fda960db60b9561f15e192c0e54a5a6a6e29ff7b86d5eb58a001489b3619adad50f4fd1f9953f39d8d7c21f20c98c8a9767f
-  data.tar.gz: 77be0c2cf8ae4c27d246581b8bebaaba579df69c536cb19be3c15918b59bcde1426a89a186ea64233cc4cd1485e47f605a744aac1955146380669338501c48be
+  metadata.gz: 17fcb827a161a709cc2b6cbda87ed645d076eae3cb77a004ca9044f951adbd6ad040b0452c2f9eedf986e35ed2107a5d5b6d89e498303d2301c3356ff2acdb38
+  data.tar.gz: 106f1726837501774021c23cb5136354dd17ef7ed134eba49840c60d3a6f065f4d7d7243048f4a0ede51a98b98e79a63845267d4d656f5132a5d398295e70c25

data/lib/splitclient-rb/engine/push_manager.rb

@@ -34,13 +34,7 @@ module SplitIoClient
       end
 
       def stop_sse
-        @config.logger.error('## STOP SSE')
-        @sse_handler.process_disconnect if @sse_handler.sse_client.nil?
         @sse_handler.stop
-        stop_next_token_refresh
-      end
-
-      def stop_next_token_refresh
         SplitIoClient::Helpers::ThreadHelper.stop(:schedule_next_token_refresh, @config)
       end
 
@@ -52,12 +46,9 @@ module SplitIoClient
             @config.logger.debug("schedule_next_token_refresh refresh in #{time} seconds.") if @config.debug_enabled
             sleep(time)
             @config.logger.debug('schedule_next_token_refresh starting ...') if @config.debug_enabled
-            @config.logger.error('## SCHEDULE NEXT TOKEN')
-
             @sse_handler.stop
             start_sse
           rescue StandardError => e
-            @config.logger.error("## SCHEDULE EXCEPTION #{e.inspect}")
             @config.logger.debug("schedule_next_token_refresh error: #{e.inspect}") if @config.debug_enabled
           end
         end

data/lib/splitclient-rb/engine/sync_manager.rb

@@ -67,7 +67,6 @@ module SplitIoClient
       end
 
       def process_action(action)
-        @config.logger.error("process_action #{action}")
         case action
         when Constants::PUSH_CONNECTED
           process_connected
@@ -118,8 +117,8 @@ module SplitIoClient
 
         @sse_connected.make_true
         @synchronizer.stop_periodic_fetch
-        @sse_handler.start_workers
         @synchronizer.sync_all
+        @sse_handler.start_workers
         record_telemetry(Telemetry::Domain::Constants::SYNC_MODE, SYNC_MODE_STREAMING)
       rescue StandardError => e
         @config.logger.error("process_connected error: #{e.inspect}")
@@ -132,8 +131,8 @@ module SplitIoClient
         end
 
         @sse_connected.make_false
-        @synchronizer.start_periodic_fetch
         @sse_handler.stop_workers
+        @synchronizer.start_periodic_fetch
         record_telemetry(Telemetry::Domain::Constants::SYNC_MODE, SYNC_MODE_POLLING)
 
         if reconnect

data/lib/splitclient-rb/helpers/thread_helper.rb

@@ -7,8 +7,8 @@ module SplitIoClient
         thread = config.threads[thread_sym]
 
         unless thread.nil?
-          Thread.kill thread
-          config.logger.debug("#{thread_sym} killed.") if config.debug_enabled
+          config.logger.debug("Stopping #{thread_sym} thread...") if config.debug_enabled
+          Thread.kill(thread)
         end
       rescue StandardError => e
         config.logger.error(e.inspect)

data/lib/splitclient-rb/sse/event_source/client.rb

@@ -35,7 +35,7 @@ module SplitIoClient
           @on[:action] = action
         end
 
-        def close
+        def close(action = nil)
           unless connected?
             @config.logger.error('SSEClient already disconected.') if @config.debug_enabled
             return
@@ -43,7 +43,7 @@ module SplitIoClient
 
           @connected.make_false
           @socket&.close
-          @config.logger.error('## CLOSE SSE')
+          dispatch_action(action) unless action.nil?
         rescue StandardError => e
           @config.logger.error("SSEClient close Error: #{e.inspect}")
         end
@@ -75,12 +75,10 @@ module SplitIoClient
 
         def connect_thread(latch)
           @config.threads[:connect_stream] = Thread.new do
-            @config.logger.error('## CONNECT THREAD')
             @config.logger.info('Starting connect_stream thread ...') if @config.debug_enabled
-            closed_with_exception = connect_stream(latch)
-            dispatch_action(closed_with_exception) unless closed_with_exception.nil?
+            action = connect_stream(latch)
+            dispatch_action(action) unless action.nil?
             @config.logger.info('connect_stream thread finished.') if @config.debug_enabled
-            @config.logger.error('#### DISCONNECTED')
           end
         end
 
@@ -98,6 +96,8 @@ module SplitIoClient
               @config.logger.error(e.inspect)
               return nil
             rescue StandardError => e
+              return nil if ENV['SPLITCLIENT_ENV'] == 'test'
+
               @config.logger.error("Error reading partial data: #{e.inspect}") if @config.debug_enabled
               return Constants::PUSH_RETRYABLE_ERROR
             end

data/lib/splitclient-rb/sse/sse_handler.rb

@@ -31,7 +31,7 @@ module SplitIoClient
       end
 
       def stop
-        @sse_client.close
+        @sse_client.close(Constants::PUSH_NONRETRYABLE_ERROR)
         stop_workers
       rescue StandardError => e
         @config.logger.debug("SSEHandler stop error: #{e.inspect}") if @config.debug_enabled

data/lib/splitclient-rb/sse/workers/segments_worker.rb

@@ -13,11 +13,6 @@ module SplitIoClient
         end
 
         def add_to_queue(change_number, segment_name)
-          unless @running.value
-            @config.logger.debug('segments worker not running.')
-            return
-          end
-
           item = { change_number: change_number, segment_name: segment_name }
           @config.logger.debug("SegmentsWorker add to queue #{item}")
           @queue.push(item)

data/lib/splitclient-rb/sse/workers/splits_worker.rb

@@ -33,21 +33,11 @@ module SplitIoClient
         end
 
         def add_to_queue(change_number)
-          unless @running.value
-            @config.logger.debug('splits worker not running.')
-            return
-          end
-
           @config.logger.debug("SplitsWorker add to queue #{change_number}")
           @queue.push(change_number)
         end
 
         def kill_split(change_number, split_name, default_treatment)
-          unless @running.value
-            @config.logger.debug('splits worker not running.')
-            return
-          end
-
           return if @splits_repository.get_change_number.to_i > change_number
 
           @config.logger.debug("SplitsWorker kill #{split_name}, #{change_number}")

data/lib/splitclient-rb/version.rb

@@ -1,3 +1,3 @@
 module SplitIoClient
-  VERSION = '7.3.3.pre.rc14'
+  VERSION = '7.3.3.pre.rc18'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: splitclient-rb
 version: !ruby/object:Gem::Version
-  version: 7.3.3.pre.rc14
+  version: 7.3.3.pre.rc18
 platform: ruby
 authors:
 - Split Software
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-01-24 00:00:00.000000000 Z
+date: 2022-01-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: allocation_stats