splitclient-rb 7.1.0.pre.rc6 → 7.1.0.pre.rc7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ab6a19db3a69d0cc5ad230bd3ed40197bd5a3af6b03b68e9cad713588bef6cc1
-  data.tar.gz: a28f0cafb9b11c5e416743b5d047e686ecc291be750489bb2496fbeca8c54be0
+  metadata.gz: d5eb756de4960f089df823c0fa7c554ce7e8a7ce2fc44dc18b0d7bf56a666ea1
+  data.tar.gz: '09f7a93cff4759f75ce467ac1d7e3853142397c4c840de46509335bd3110a3f3'
 SHA512:
-  metadata.gz: f62894b96a044d8a2a96be82f00af0c6533b585ca856b67dcbd2f65806dbb3652fa0c767ba7d6cf8fa1e9c18444bf966406758ca6e12af5059b10ba46e35931a
-  data.tar.gz: 9156b2aa0c262e3595b86daf1632feeece3f9dd85d041becd3df3907e1cfa0c719d86cbc17dc1f177c72dcd35e1599b845a27e87c3cc33ab1e73e8c62b5dc93d
+  metadata.gz: fa93b0737066384f8e2e49bab166f8b23057578b6f1b940c9a27ff64cbcada99b3a982cbb6d43f78268d820189064c0b554761b84c3b198596eff6d0fc92fa20
+  data.tar.gz: 6dcac7c1907c25d322a004a290682e383cdc2f3e6880d25b2df5731a89beba7900ef0404eaffee895e6bbe3618315e32fe8bba0c14796f64ca96073088e0b773

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

@@ -37,10 +37,16 @@ module SplitIoClient
 
       def schedule_next_token_refresh(time)
         @config.threads[:schedule_next_token_refresh] = Thread.new do
-          @config.logger.debug("schedule_next_token_refresh refresh in #{time} seconds.") if @config.debug_enabled
-          sleep(time)
-          stop_sse
-          start_sse
+          begin
+            @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
+            stop_sse
+            sleep(1)
+            start_sse
+          rescue StandardError => e
+            @config.logger.debug("schedule_next_token_refresh error: #{e.inspect}") if @config.debug_enabled
+          end
         end
       end
     end

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

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

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

@@ -11,8 +11,7 @@ module SplitIoClient
         DEFAULT_READ_TIMEOUT = 70
         KEEP_ALIVE_RESPONSE = "c\r\n:keepalive\n\n\r\n".freeze
 
-        def initialize(url, config, read_timeout: DEFAULT_READ_TIMEOUT)
-          @uri = URI(url)
+        def initialize(config, read_timeout: DEFAULT_READ_TIMEOUT)
           @config = config
           @read_timeout = read_timeout
           @connected = Concurrent::AtomicBoolean.new(false)
@@ -22,9 +21,6 @@ module SplitIoClient
           @on = { event: ->(_) {}, connected: ->(_) {}, disconnect: ->(_) {} }
 
           yield self if block_given?
-
-          connect_thread
-          connect_passenger_forked if defined?(PhusionPassenger)
         end
 
         def on_event(&action)
@@ -42,8 +38,17 @@ module SplitIoClient
         def close
           dispatch_disconnect
           @connected.make_false
+          SplitIoClient::Helpers::ThreadHelper.stop(:connect_stream, @config)
           @socket&.close
           @socket = nil
+          @config.threads[:connect_stream] = nil
+        end
+
+        def start(url)
+          @uri = URI(url)
+
+          connect_thread
+          connect_passenger_forked if defined?(PhusionPassenger)
         end
 
         def connected?
@@ -53,7 +58,10 @@ module SplitIoClient
         private
 
         def connect_thread
-          @config.threads[:connect_stream] = Thread.new { connect_stream }
+          @config.threads[:connect_stream] = Thread.new do
+            @config.logger.info('Starting connect_stream thread ...') if @config.debug_enabled
+            connect_stream
+          end
         end
 
         def connect_passenger_forked
@@ -70,17 +78,13 @@ module SplitIoClient
 
           while @connected.value
             begin
-              partial_data = @socket.readpartial(10000, timeout: @read_timeout)
-            rescue Socketry::TimeoutError
-              @config.logger.error("Socket read time out in #{@read_timeout} seconds") if @config.debug_enabled
+              partial_data = @socket.readpartial(10_000, timeout: @read_timeout)
+            rescue StandardError => e
+              @config.logger.error(e.inspect) if @config.debug_enabled
               @connected.make_false
               @socket&.close
               @socket = nil
               connect_stream
-            rescue StandardError => e
-              @config.logger.error(e.inspect)
-              close
-              connect_stream
             end
 
             process_data(partial_data)
@@ -148,7 +152,7 @@ module SplitIoClient
 
           events
         rescue StandardError => e
-          @config.logger.debug(buffer)
+          @config.logger.error(buffer)
           @config.logger.error("Error during parsing a event: #{e.inspect}")
           []
         end

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

@@ -11,6 +11,11 @@ module SplitIoClient
         @splits_worker = SplitIoClient::SSE::Workers::SplitsWorker.new(synchronizer, config, splits_repository)
         @segments_worker = SplitIoClient::SSE::Workers::SegmentsWorker.new(synchronizer, config, segments_repository)
         @notification_processor = SplitIoClient::SSE::NotificationProcessor.new(config, @splits_worker, @segments_worker)
+        @sse_client = SSE::EventSource::Client.new(@config) do |client|
+          client.on_event { |event| handle_incoming_message(event) }
+          client.on_connected { process_connected }
+          client.on_disconnect { process_disconnect }
+        end
 
         @on = { connected: ->(_) {}, disconnect: ->(_) {} }
 
@@ -19,17 +24,14 @@ module SplitIoClient
 
       def start(token_jwt, channels)
         url = "#{@config.streaming_service_url}?channels=#{channels}&v=1.1&accessToken=#{token_jwt}"
-
-        @sse_client = SSE::EventSource::Client.new(url, @config) do |client|
-          client.on_event { |event| handle_incoming_message(event) }
-          client.on_connected { process_connected }
-          client.on_disconnect { process_disconnect }
-        end
+        @sse_client.start(url)
       end
 
       def stop
-        @sse_client&.close
-        @sse_client = nil
+        @sse_client.close
+        stop_workers
+      rescue StandardError => e
+        @config.logger.debug("SSEHandler stop error: #{e.inspect}") if @config.debug_enabled
       end
 
       def connected?

data/lib/splitclient-rb/version.rb

@@ -1,3 +1,3 @@
 module SplitIoClient
-  VERSION = '7.1.0.pre.rc6'
+  VERSION = '7.1.0.pre.rc7'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: splitclient-rb
 version: !ruby/object:Gem::Version
-  version: 7.1.0.pre.rc6
+  version: 7.1.0.pre.rc7
 platform: ruby
 authors:
 - Split Software
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-04-23 00:00:00.000000000 Z
+date: 2020-04-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: allocation_stats