Move error handling to ensure loop continues after child process management (#46)

Author: macournoyer

lib/async/container/generic.rb

@@ -191,6 +191,8 @@ def spawn(name: nil, restart: false, key: nil, health_check_timeout: nil, &block
 							age_clock = state[:age] = Clock.start
 						end
 
+						status = nil
+						
 						begin
 							status = @group.wait_for(child) do |message|
 								case message
@@ -203,11 +205,13 @@ def spawn(name: nil, restart: false, key: nil, health_check_timeout: nil, &block
 									age_clock&.reset!
 								end
 							end
+						rescue => error
+							Console.error(self, "Error during child process management!", exception: error, running: @running)
 						ensure
 							delete(key, child)
 						end
 
-						if status.success?
+						if status&.success?
 							Console.info(self, "Child exited successfully.", status: status, running: @running)
 						else
 							@statistics.failure!
@@ -220,9 +224,6 @@ def spawn(name: nil, restart: false, key: nil, health_check_timeout: nil, &block
 							break
 						end
 					end
-				rescue => error
-					Console.error(self, "Failure during child process management!", exception: error, running: @running)
-					raise
 				ensure
 					Console.info(self, "Child process management loop exited.", running: @running)
 				end.resume

releases.md

@@ -1,5 +1,9 @@
 # Releases
 
+## Unreleased
+
+  - Make the child handling more robust in the face of exceptions.
+
 ## v0.27.4
 
   - Fix race condition where `wait_for` could modify `@running` while it was being iterated over (`each_value`) during health checks.