splitclient-rb 8.11.1.pre.rc1 → 8.11.1.pre.rc2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 936f987bddefefa15d687d9fd3a772d10196e2174fee301b3b86e32f11788b2d
-  data.tar.gz: 2869f78cc88aaf326af8b0f9a59f3837f69e47ee26238ed9de7061ffb7173130
+  metadata.gz: 6b8826705cc51c97b0fd56af17ba520dbc20932c4c658c3e55f0ec04cd9fc850
+  data.tar.gz: 3eb1a3e21d490fe80b80f67f73e3372bb822a15607f2231ca3fe335f37cb49ae
 SHA512:
-  metadata.gz: f33ee7d407f785a61a3f510ae55fc1874e63cfb62d4543d260b4daaaeab18f62e3d9fd02c44ba88a554f8be09498f904438243410b8d37a73dc9de68e8bc75ed
-  data.tar.gz: 8ba5b3941841ec950c4e51149733edb33b1685cbf1091c772f23ff837829793aeb105da48bbaf5cc5c20342243146ecc76b865f13d16ca9c855789dfa5df02c7
+  metadata.gz: 2abb05c726b7019e7f56240f5b6e69d50689fae3739e714e85c7134ff47fec1bffd54b68bca6c571162a590e87e87ae090376ad7b9854f65373060ddb532bd93
+  data.tar.gz: b80cbe9440ec9bf0ff54b057bd95b5c248da1781d1eea157ef9baaa8ff473826c280152e45a2c41b41e8f49dc219733234671984f715b5a7119a37f5dba71b9b

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

@@ -16,17 +16,21 @@ module SplitIoClient
         response = @auth_api_client.authenticate(@api_key)
         @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])
-          @back_off.reset
-          record_telemetry(response[:exp])
+        unless response[:push_enabled]
+          schedule_next_token_refresh(@back_off.interval) if response[:retry]
+          return false
+        end
 
-          return true
+        unless @sse_handler.start(response[:token], response[:channels])
+          @config.logger.debug('Streaming server returned error') if @config.debug_enabled
+          stop_sse
+          return false
         end
 
-        stop_sse
-        schedule_next_token_refresh(@back_off.interval) if response[:retry]
-        false
+        schedule_next_token_refresh(response[:exp])
+        @back_off.reset
+        record_telemetry(response[:exp])
+        true
       rescue StandardError => e
         @config.logger.error("start_sse: #{e.inspect}")
       end

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

@@ -37,17 +37,15 @@ module SplitIoClient
         end
 
         def close(status = nil)
-          unless connected?
-            @config.logger.debug('SSEClient already disconected.') if @config.debug_enabled
-            return
-          end
-          @config.logger.debug("Closing SSEClient socket") if @config.debug_enabled
+          return if @socket.nil?
 
+          @config.logger.debug("Closing SSEClient socket") if @config.debug_enabled
           push_status(status)
           @connected.make_false
           @socket.sync_close = true if @socket.is_a? OpenSSL::SSL::SSLSocket
           @socket.close
           @config.logger.debug("SSEClient socket state #{@socket.state}") if @socket.is_a?(OpenSSL::SSL::SSLSocket) && @config.debug_enabled
+          @socket = nil
         rescue StandardError => e
           @config.logger.error("SSEClient close Error: #{e.inspect}")
         end
@@ -80,7 +78,7 @@ module SplitIoClient
           @config.threads[:connect_stream] = Thread.new do
             @config.logger.info('Starting connect_stream thread ...')
             new_status = connect_stream(latch)
-            push_status(new_status)
+            push_status(new_status) unless new_status.nil?
             @config.logger.info('connect_stream thread finished.')
           end
         end
@@ -93,11 +91,6 @@ module SplitIoClient
                 begin
                   partial_data = @socket.readpartial(10_000)
                   read_first_event(partial_data, latch)
-
-                  if partial_data == :eof
-                    @config.logger.error("SSE recived EOF unexpectedly")
-                    return Constants::PUSH_RETRYABLE_ERROR
-                  end
                 rescue IO::WaitReadable => e
                   @config.logger.debug("SSE client IO::WaitReadable transient error: #{e.inspect}") if @config.debug_enabled
                   IO.select([@socket], nil, nil, @read_timeout)
@@ -110,7 +103,7 @@ module SplitIoClient
                   @config.logger.error("SSE read operation timed out!: #{e.inspect}")
                   return Constants::PUSH_RETRYABLE_ERROR
                 rescue EOFError => e
-                  @config.logger.error("SSE read operation EOF Exception!: #{e.inspect}")
+                  @config.logger.error("SSE read operation EOF, server closed the connection, will reconnect: #{e.inspect}")
                   return Constants::PUSH_RETRYABLE_ERROR
                 rescue Errno::EBADF, IOError => e
                   @config.logger.error("SSE read operation EBADF or IOError: #{e.inspect}")

data/lib/splitclient-rb/version.rb

@@ -1,3 +1,3 @@
 module SplitIoClient
-  VERSION = '8.11.1-rc1'
+  VERSION = '8.11.1-rc2'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: splitclient-rb
 version: !ruby/object:Gem::Version
-  version: 8.11.1.pre.rc1
+  version: 8.11.1.pre.rc2
 platform: ruby
 authors:
 - Split Software