splitclient-rb 7.3.3.pre.rc9 → 7.3.3.pre.rc13

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ec6fc8163dee8d303228ac119e0d704bd827b00c2a4dd6408cef7af1c7f35767
-  data.tar.gz: 841be300c51ec06e5abc3b916ab69efeb42bfb15efc36179673dcefce7fd2165
+  metadata.gz: ea33a02d9d4529927792f590796263d2eb2985ef367183acdb3a8be92e92b518
+  data.tar.gz: d7fca9070abddd5347600d4be81411d253efcbfb3c42a3dd5a929c718aae6b08
 SHA512:
-  metadata.gz: 2fa16a7e50e717f9f0af64d8159a586ca9ece0dcba1c4f56914f73064a07c2fe30edc70ce9460fab3e5cd9282885179a4a4479dcbc613e569c926e9c31d09b57
-  data.tar.gz: b6e45d41dc353aeca5cb98f666814d57d9481cb8844fdd7170647dead43071a8fbe76d30a7ee45aceb8f4dd43c317bb5806dd20a4e538d64e7e888c0f50c0a81
+  metadata.gz: b565bb319c60aa4c6589d9480940fedf7ee46e44bf2f43062d4f887059d21be2330802e91a18d0acb094d8b9c8721f704318f1e934ed537c2d7cecbb326f1244
+  data.tar.gz: 28f349e55ab2067ba016115a5ee09f730b7c470c99fca4ec84ce00ad45ecb0caadbab8641ab504f05a8926040fc8e050c3a6b14df765cc77ace3f2de347b7314

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

@@ -18,7 +18,7 @@ module SplitIoClient
         @config.logger.debug("Auth service response push_enabled: #{response[:push_enabled]}") if @config.debug_enabled
 
         if response[:push_enabled] && @sse_handler.start(response[:token], response[:channels])
-          schedule_next_token_refresh(response[:exp])
+          schedule_next_token_refresh(150)
           @back_off.reset
           record_telemetry(response[:exp])
 
@@ -34,8 +34,13 @@ 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
 
@@ -47,9 +52,12 @@ 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,6 +67,7 @@ module SplitIoClient
       end
 
       def process_action(action)
+        @config.logger.error("process_action #{action}")
         case action
         when Constants::PUSH_CONNECTED
           process_connected
@@ -117,8 +118,8 @@ module SplitIoClient
 
         @sse_connected.make_true
         @synchronizer.stop_periodic_fetch
-        @synchronizer.sync_all
         @sse_handler.start_workers
+        @synchronizer.sync_all
         record_telemetry(Telemetry::Domain::Constants::SYNC_MODE, SYNC_MODE_STREAMING)
       rescue StandardError => e
         @config.logger.error("process_connected error: #{e.inspect}")
@@ -131,11 +132,12 @@ module SplitIoClient
         end
 
         @sse_connected.make_false
-        @sse_handler.stop_workers
         @synchronizer.start_periodic_fetch
+        @sse_handler.stop_workers
         record_telemetry(Telemetry::Domain::Constants::SYNC_MODE, SYNC_MODE_POLLING)
 
         if reconnect
+          @push_manager.stop_sse
           @synchronizer.sync_all
           @push_manager.start_sse
         end

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

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

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

@@ -35,11 +35,15 @@ module SplitIoClient
           @on[:action] = action
         end
 
-        def close(action = Constants::PUSH_NONRETRYABLE_ERROR)
+        def close
+          unless connected?
+            @config.logger.error('SSEClient already disconected.') if @config.debug_enabled
+            return
+          end
+
           @connected.make_false
-          SplitIoClient::Helpers::ThreadHelper.stop(:connect_stream, @config)
           @socket&.close
-          dispatch_action(action)
+          @config.logger.error('## CLOSE SSE')
         rescue StandardError => e
           @config.logger.error("SSEClient close Error: #{e.inspect}")
         end
@@ -71,13 +75,17 @@ 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
-            connect_stream(latch)
+            closed_with_exception = connect_stream(latch)
+            dispatch_action(closed_with_exception) unless closed_with_exception.nil?
+            @config.logger.info('connect_stream thread finished.') if @config.debug_enabled
+            @config.logger.error('#### DISCONNECTED')
           end
         end
 
         def connect_stream(latch)
-          socket_write(latch)
+          return Constants::PUSH_NONRETRYABLE_ERROR unless socket_write(latch)
 
           while connected? || @first_event.value
             begin
@@ -86,24 +94,28 @@ module SplitIoClient
               read_first_event(partial_data, latch)
 
               raise 'eof exception' if partial_data == :eof
+            rescue Errno::EBADF, IOError => e
+              @config.logger.error(e.inspect)
+              return nil
             rescue StandardError => e
-              @config.logger.error('Error reading partial data: ' + e.inspect) if @config.debug_enabled
-              close(Constants::PUSH_RETRYABLE_ERROR)
-              return
+              @config.logger.error("Error reading partial data: #{e.inspect}") if @config.debug_enabled
+              return Constants::PUSH_RETRYABLE_ERROR
             end
 
             process_data(partial_data)
           end
+          nil
         end
 
         def socket_write(latch)
           @first_event.make_true
           @socket = socket_connect
           @socket.write(build_request(@uri))
+          true
         rescue StandardError => e
           @config.logger.error("Error during connecting to #{@uri.host}. Error: #{e.inspect}")
-          close(Constants::PUSH_NONRETRYABLE_ERROR)
           latch.count_down
+          false
         end
 
         def read_first_event(data, latch)
@@ -181,8 +193,10 @@ module SplitIoClient
         end
 
         def dispatch_action(action)
-          @config.logger.debug("Dispatching action: #{action}") if @config.debug_enabled
-          @on[:action].call(action)
+          @config.threads[:dispatch_action] = Thread.new do
+            @config.logger.debug("Dispatching action: #{action}") if @config.debug_enabled
+            @on[:action].call(action)
+          end
         end
       end
     end

data/lib/splitclient-rb/version.rb

@@ -1,3 +1,3 @@
 module SplitIoClient
-  VERSION = '7.3.3.pre.rc9'
+  VERSION = '7.3.3.pre.rc13'
 end

metadata

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