nobject 1.1.18 → 1.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 735417071860a888e86bb652a14f147bdca25d66add3b2cb6d09509e4f6c05ad
-  data.tar.gz: d5462edfd3224e520907f0b7c3323ff00bb3654cf4103c3170c8f0e5074f3c51
+  metadata.gz: e47a1b1add27980c6649b852026a4393dd7b708a815bc1d395c8e918b3c2a375
+  data.tar.gz: 3caa113ae4fc833f1fdd6b0505b94a52b94f2ab60081b315132f9dabf1e5f008
 SHA512:
-  metadata.gz: 8096bc14015f9604f9b594844cf3c96b2a1452a89c93ef0334d921a6cc0d9ffc6caf5f6dc5e8f98065ae9ab2568a3a1297002e0724af279c3f97d9b4df6d2c4b
-  data.tar.gz: ecbd98324b162bbb7b7bd2ed051411a06b5df8334717282e3ac9c8cf53f1a90f56840055865242580b16e368f4aa5eb89a1cd4b75f60a20c6a45f62d69fc1e59
+  metadata.gz: 611d074f8fed895c95644147edc3620fbd3e0fe23485e0abf45426a1a1895576f03f17eb7d7d1b4ffd230bc2256e23fff0e4f07fe5705cbedab0868f4c680d48
+  data.tar.gz: b80d4f3f3def8cadbfd17e2bc8bb7a71ae1f648f1b62414fe6b87044e6e80c3f8f1030961063691092e4ca6d6a95179e654b0ee35bbe8c56376637e641d4a61a

data/lib/nobject/local.rb

@@ -5,6 +5,8 @@ module Nobject
   # it to a Nobject::Serve4r, which will then  send method calls to a matching
   # Nobject::Remote object
   class Local
+    MAX_RETRIES = 3
+
     # host: the hostname of the server to push obj to
     # port: the port number of the server to push obj to
     # obj: the obj to store over the network
@@ -29,36 +31,55 @@ module Nobject
       msg = { method: method, args: args }
       msg_bytes = Marshal.dump(msg)
 
-      begin
-        @socket.send([msg_bytes.length].pack('Q>'), 0)
-        File.open('/tmp/nobject.log', 'a') {|f| f.puts "  LMS:##{@msg_counter += 1} sz#{msg_bytes.length} m:#{method}"; f.flush }
-        @socket.send(msg_bytes, 0)
-        @socket.flush
-      rescue Exception
-        raise Local::MethodRequestFailure.new("did not receive response from call to `#{method}' over the network")
-      end
+      retries = 0
+      loop do
+        raise MethodInvocationRetriesExceeded.new("Exceeded retry limit (#{MAX_RETRIES})") if retries == MAX_RETRIES
+
+        begin
+          @socket.send([msg_bytes.length].pack('Q>'), 0)
+          File.open('/tmp/nobject.log', 'a') {|f| f.puts "  LMS:##{@msg_counter += 1} sz#{msg_bytes.length} m:#{method}"; f.flush }
+          @socket.send(msg_bytes, 0)
+          @socket.flush
+        rescue Exception
+          raise Local::MethodRequestFailure.new("did not receive response from call to `#{method}' over the network")
+        end
+
+        return_data = begin
+                        msg_size = @socket.recv(8).unpack('Q>').first
+                        raw_bytes = @socket.recv(msg_size)
+                        File.open('/tmp/nobject.log', 'a') {|f| f.puts "    LMGotit :##{@msg_counter += 1} sz#{msg_size} bytes:#{raw_bytes.length} m:#{method}"; f.flush }
+                        if msg_size != raw_bytes.length
+                          retries += 1
+                          File.open('/tmp/nobject.log', 'a') {|f| f.puts "    LMGotitRETRY"; f.flush }
+                          redo
+                        end
 
-      return_data = begin
-                      msg_size = @socket.recv(8).unpack('Q>').first
-                      raw_bytes = @socket.recv(msg_size)
-                      File.open('/tmp/nobject.log', 'a') {|f| f.puts "    LMGotit :##{@msg_counter += 1} sz#{msg_size} bytes:#{raw_bytes.length} m:#{method}"; f.flush }
-                      Marshal.load(raw_bytes)
-                    rescue Exception => e
-                      error_msg = <<~MSG
-                        did not receive response from call to `#{method}' over the network
-                        would have been msg_id #{@msg_counter} OR #{@msg_counter + 1} when trying to receive #{msg_size} bytes
-                        caused by #{e.class.name}
-                          exception backtrace:
-                          #{e.backtrace.join("\n    ")}
-                      MSG
-                      raise Local::MethodResponseFailure.new(error_msg)
-                    end
+                        Marshal.load(raw_bytes)
+                      rescue Exception => e
+                        # TODO: consider removing this rescue block:
+                        #   this rescue may be unreachable now, since the only
+                        #   time it used to happen was when:
+                        #     msg_size != raw_bytes.length
+                        #
+                        #   ... which is now handled by the retry logic above
+                        error_msg = <<~MSG
+                          did not receive response from call to `#{method}' over the network
+                          would have been msg_id #{@msg_counter} OR #{@msg_counter + 1} when trying to receive #{msg_size} bytes
+                          caused by #{e.class.name}
+                            exception backtrace:
+                            #{e.backtrace.join("\n    ")}
+                        MSG
+                        raise Local::MethodResponseFailure.new(error_msg)
+                      end
 
-      case return_data.first
-      when :ok then return_data.last
-      when :raise then raise return_data.last
-      else
-        raise Local::UnknownReturnDataType.new("unknown data type '#{return_data.first}' within Nobject::Local (Nobject::Local::UnknownReturnDataType)")
+        break (
+          case return_data.first
+          when :ok then return_data.last
+          when :raise then raise return_data.last
+          else
+            raise Local::UnknownReturnDataType.new("unknown data type '#{return_data.first}' within Nobject::Local (Nobject::Local::UnknownReturnDataType)")
+          end
+        )
       end
     end
 
@@ -77,4 +98,5 @@ module Nobject
   class Local::InvalidMethod < RuntimeError; end
   class Local::MethodRequestFailure < RuntimeError; end
   class Local::MethodResponseFailure < RuntimeError; end
+  class Local::MethodInvocationRetriesExceeded < RuntimeError; end
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: nobject
 version: !ruby/object:Gem::Version
-  version: 1.1.18
+  version: 1.2.1
 platform: ruby
 authors:
 - Jeff Lunt
 bindir: bin
 cert_chain: []
-date: 2025-05-01 00:00:00.000000000 Z
+date: 2025-05-08 00:00:00.000000000 Z
 dependencies: []
 description: network-hosted objects that you can call methods on as if they were local
 email: jefflunt@gmail.com