async-container 0.34.4 → 0.34.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 45d1096677e99abf4e408324f81089d3a09dffe06faec9802505f0982a286c9c
-  data.tar.gz: 9297103dfdd8c2181881a7cc6639afcc4d16ea3acfcd4f6074429c6a06c229f5
+  metadata.gz: 277b861e3816090ba6221ede785567b38c4e4f379c191bdbf96134245b6bd618
+  data.tar.gz: 6c173b66aa334bd493f946027437f3947fd4b3c85f4b05e4c80bccb0bfe2dc72
 SHA512:
-  metadata.gz: c1e6e6c690ab46402878c54a91565decb35a5cfbb6f10b49e393949fafe2cba33da780bbdbb9a2b01dd9ad80487a16e83c7188667f22ee6e8c8696008a0e8ce2
-  data.tar.gz: fce543a64ebb908baa70bd0c5912b92b1395e3a99fc257cc3280cd3527f23215001e1aebfa50946f6fd956011f981b84e92a8ced7d4079e063fa9be542d1e8c6
+  metadata.gz: 18613e589fd52228a3d4b99c53ad2e7a493abf203e9b310c94bb576db5b78a424da16bd96be5ee4faa6cfa11ed737388fba1d87fe63e2a182afd9f3bf193d954
+  data.tar.gz: fef669494da3b2b43c0a4a4ec7b43394c059842e0522279816a7e5d2fa1731bbaafd11e66f792672e93c238589e05874b90ab22fd6af0a9aca02f16a9f5f3cfe

checksums.yaml.gz.sig

@@ -1,6 +1,3 @@
-Ih�O��11���o׈��/ֹ�?�'�L�ܛ5��z�Nۧ0c��v>q��(�vG�V�&F��e�L����ʘu�E���P!ġ�l}ѻ��TaRo9a��������D��)p��?�
-e�¼���
-�m�4�kR�N
-�.����J!��Ɲ�L3_C�� V����J�����uF���.���"'}J�G�|��cQk�䬽@^	��0-()�>&��1�O^��3�����+���)�,����JGAJ�4�g�p	���5.���rc��-�I9Z�-��r�x�c靔��[��� ���H��
-��3�Ҵ�^���
-�Jh\,��(S�ɸ!��l���|��r�
K�V�������
+���c����y��nIuN��?��K�Z�m֍��I�2�qP�P�Ο�D+���
+��O��[�4�"�ɿw� ��'fR8�V'��Q;u�K��xy�w� Yɥ�׋b�N��h����6R���s�֨����-�g�(1�o?�d�e����=$OYv�-�7縦W8�f^�񘰀eI��ư9�H�3�A|̂�����]�W�ь�]����i�6���\�X�䶎�.֤�Ίʫ h�)F|����N����^7���U{�7>K���M�VI�o�C�/�%�kn�@�i�vכ&R|�����~�3
+��/�qu���Yi�^�ֹ�.��+qbRR�w�\�

data/lib/async/container/forked.rb

@@ -80,13 +80,15 @@ module Async
 					# This method replaces the child process with the new executable, thus this method never returns.
 					#
 					# @parameter arguments [Array] The arguments to pass to the new process.
-					# @parameter ready [Boolean] If true, informs the parent process that the child is ready. Otherwise, the child process will need to use a notification protocol to inform the parent process that it is ready.
+					# @parameter ready [Boolean] If true, informs the parent process that the child is ready before exec. The notification pipe will still be passed to the exec'd process to prevent premature termination.
 					# @parameter options [Hash] Additional options to pass to {::Process.exec}.
 					def exec(*arguments, ready: true, **options)
+						# Always set up the notification pipe to be inherited by the exec'd process.
+						# This prevents the pipe from closing, which would trigger hang prevention and SIGKILL.
+						self.before_spawn(arguments, options)
+						
 						if ready
 							self.ready!(status: "(exec)")
-						else
-							self.before_spawn(arguments, options)
 						end
 						
 						::Process.exec(*arguments, **options)

data/lib/async/container/threaded.rb

@@ -91,10 +91,11 @@ module Async
 					# Execute a child process using {::Process.spawn}. In order to simulate {::Process.exec}, an {Exit} instance is raised to propagage exit status.
 					# This creates the illusion that this method does not return (normally).
 					def exec(*arguments, ready: true, **options)
+						# Always set up the notification pipe to be inherited by the spawned process.
+						self.before_spawn(arguments, options)
+						
 						if ready
 							self.ready!(status: "(spawn)")
-						else
-							self.before_spawn(arguments, options)
 						end
 						
 						begin

data/lib/async/container/version.rb

@@ -5,6 +5,6 @@
 
 module Async
 	module Container
-		VERSION = "0.34.4"
+		VERSION = "0.34.5"
 	end
 end

data/readme.md

@@ -28,6 +28,10 @@ Please see the [project documentation](https://socketry.github.io/async-containe
 
 Please see the [project releases](https://socketry.github.io/async-container/releases/index) for all releases.
 
+### v0.34.5
+
+  - **Fixed**: `instance.exec` with `ready: true` no longer causes premature termination. The notification pipe is now always passed to the exec'd process.
+
 ### v0.34.4
 
   - Add missing `bake` and `context` files to the release.
@@ -68,10 +72,6 @@ Please see the [project releases](https://socketry.github.io/async-container/rel
   - `SIGTERM` is now graceful, the same as `SIGINT`, for better compatibility with Kubernetes and systemd.
   - `ASYNC_CONTAINER_INTERRUPT_TIMEOUT` and `ASYNC_CONTAINER_TERMINATE_TIMEOUT` are removed and replaced by `ASYNC_CONTAINER_GRACEFUL_TIMEOUT`.
 
-### v0.29.0
-
-  - Introduce `Client#healthy!` for sending health check messages.
-
 ## Contributing
 
 We welcome contributions to this project.

data/releases.md

@@ -1,5 +1,9 @@
 # Releases
 
+## v0.34.5
+
+  - **Fixed**: `instance.exec` with `ready: true` no longer causes premature termination. The notification pipe is now always passed to the exec'd process.
+
 ## v0.34.4
 
   - Add missing `bake` and `context` files to the release.

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: async-container
 version: !ruby/object:Gem::Version
-  version: 0.34.4
+  version: 0.34.5
 platform: ruby
 authors:
 - Samuel Williams