nobject 1.1.18 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 735417071860a888e86bb652a14f147bdca25d66add3b2cb6d09509e4f6c05ad
-  data.tar.gz: d5462edfd3224e520907f0b7c3323ff00bb3654cf4103c3170c8f0e5074f3c51
+  metadata.gz: 17e292f75e9f260c1c4d1183f5ad12749fa74b98f5de313ee89cb8c9bfe99935
+  data.tar.gz: a83497c75a0280157b671fd84005c0ea0410d25e5fec4b535b6db26389c110c3
 SHA512:
-  metadata.gz: 8096bc14015f9604f9b594844cf3c96b2a1452a89c93ef0334d921a6cc0d9ffc6caf5f6dc5e8f98065ae9ab2568a3a1297002e0724af279c3f97d9b4df6d2c4b
-  data.tar.gz: ecbd98324b162bbb7b7bd2ed051411a06b5df8334717282e3ac9c8cf53f1a90f56840055865242580b16e368f4aa5eb89a1cd4b75f60a20c6a45f62d69fc1e59
+  metadata.gz: 6b3a073baec24f5cf52d1e6100e6104c646efe0f6de803dd02f2833979de36b9ae7d3bbedc10cab942515e636f7b41f359e5617c6d0404159316116b41d71cea
+  data.tar.gz: 22f333104e7546330883519361c80c4b287bb3a8402d647ebee0dc39ea42e6523b8c876ee8d6286eee609d9b85710b4b3ea2d6b96a4aab52ee152149e648e1ff

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,57 @@ 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
+                          print "\a"  # TODO: consider removing this after the
+                                      #   retry logic seems solid, it's extra,
+                                      #   literal noise :)
+                          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 +100,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.0
 platform: ruby
 authors:
 - Jeff Lunt
 bindir: bin
 cert_chain: []
-date: 2025-05-01 00:00:00.000000000 Z
+date: 2025-05-02 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