async-http 0.78.0 → 0.80.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 76e78af8d20808d557b1c13c850af0a268de5535726ba07708b2012eb8752029
-  data.tar.gz: 53bdf9b950394a03ca8c1fd47befe51e8ba51a5cce82aa47c01111df7efd21b4
+  metadata.gz: 5a73103da274cc4cac5838409d95a05f1c8a746e0b58dcee062031886061ffb7
+  data.tar.gz: da1d5991f78d71c2ab1a3c8ace1d0d8c53831b545bdcfe2f3e93d095bd6389bd
 SHA512:
-  metadata.gz: 43c49b50bb88955088b74926e3fe33310f38f4b09d9b9ecdc538a9835c9be5b8be0e172e09794324a53db99e09b0102ae96fc179af13578a758f42644d6f4011
-  data.tar.gz: fa1a0d97f326bdd72010f07cfe21004b0da8f75fe4556778bd3b918f42e2e85f22bc41cd07e2a64b4395540e531a879480e634a8fe401ee0d0d7c146af8b61d7
+  metadata.gz: 4169f9f110a07db5f27b6e083b09b1c6d3f0940269b1d423150993a491c4f3d46dfa4ae227712502b07d4cf949b0186e1e838fa3b77cead2809ee73c30e4500c
+  data.tar.gz: a85d33bd88881433c1c33b369027d6ff4f1c095b26ecac5b583b865fe2f8011c6ee7bba6c6f9857c5b1cf1697c4298a5e4c6506da3584d0215682bd4beea9858

data/lib/async/http/protocol/http1/client.rb

@@ -18,11 +18,12 @@ module Async
 					
 					attr_accessor :pool
 					
-					def closed!
+					def closed(error = nil)
 						super
 						
 						if pool = @pool
 							@pool = nil
+							# If the connection is not reusable, this will retire it from the connection pool and invoke `#close`.
 							pool.release(self)
 						end
 					end
@@ -50,30 +51,32 @@ module Async
 								task.async(annotation: "Upgrading request...") do
 									# If this fails, this connection will be closed.
 									write_upgrade_body(protocol, body)
+								rescue => error
+									self.close(error)
 								end
 							elsif request.connect?
 								task.async(annotation: "Tunnneling request...") do
 									write_tunnel_body(@version, body)
+								rescue => error
+									self.close(error)
 								end
 							else
 								task.async(annotation: "Streaming request...") do
 									# Once we start writing the body, we can't recover if the request fails. That's because the body might be generated dynamically, streaming, etc.
 									write_body(@version, body, false, trailer)
+								rescue => error
+									self.close(error)
 								end
 							end
 						elsif protocol = request.protocol
 							write_upgrade_body(protocol)
 						else
-							write_body(@version, body, false, trailer)
+							write_body(@version, request.body, false, trailer)
 						end
 						
-						response = Response.read(self, request)
-						
-						return response
-					rescue
-						# This will ensure that #reusable? returns false.
-						self.close
-						
+						return Response.read(self, request)
+					rescue => error
+						self.close(error)
 						raise
 					end
 				end

data/lib/async/http/protocol/http1/connection.rb

@@ -43,6 +43,8 @@ module Async
 					
 					def read_line?
 						@stream.read_until(CRLF)
+					rescue Errno::ECONNRESET
+						return nil
 					end
 					
 					def read_line

data/lib/async/http/protocol/http1/finishable.rb

@@ -32,19 +32,23 @@ module Async
 					end
 					
 					def close(error = nil)
+						super
+						
 						unless @closed.resolved?
 							@error = error
 							@closed.value = true
 						end
-						
-						super
 					end
 					
-					def wait
+					def wait(persistent = true)
 						if @reading
 							@closed.wait
-						else
+						elsif persistent
+							# If the connection can be reused, let's gracefully discard the body:
 							self.discard
+						else
+							# Else, we don't care about the body, so we can close it immediately:
+							self.close
 						end
 					end
 					

data/lib/async/http/protocol/http1/server.rb

@@ -22,7 +22,7 @@ module Async
 						@ready = Async::Notification.new
 					end
 					
-					def closed!
+					def closed(error = nil)
 						super
 						
 						@ready.signal
@@ -38,14 +38,12 @@ module Async
 					end
 					
 					def next_request
-						# Wait for the connection to become idle before reading the next request:
-						unless idle?
+						if closed?
+							return nil
+						elsif !idle?
 							@ready.wait
 						end
 						
-						# The default is true.
-						return unless @persistent
-						
 						# Read an incoming request:
 						return unless request = Request.read(self)
 						
@@ -90,38 +88,41 @@ module Async
 										# We force a 101 response if the protocol is upgraded - HTTP/2 CONNECT will return 200 for success, but this won't be understood by HTTP/1 clients:
 										write_response(@version, 101, response.headers)
 										
-										stream = write_upgrade_body(protocol)
-										
 										# At this point, the request body is hijacked, so we don't want to call #finish below.
 										request = nil
 										response = nil
 										
-										# We must return here as no further request processing can be done:
-										return body.call(stream)
+										if body.stream?
+											return body.call(write_upgrade_body(protocol))
+										else
+											write_upgrade_body(protocol, body)
+										end
 									elsif response.status == 101
 										# This code path is to support legacy behavior where the response status is set to 101, but the protocol is not upgraded. This may not be a valid use case, but it is supported for compatibility. We expect the response headers to contain the `upgrade` header.
 										write_response(@version, response.status, response.headers)
 										
-										stream = write_tunnel_body(version)
-										
 										# Same as above:
 										request = nil
 										response = nil
 										
-										# We must return here as no further request processing can be done:
-										return body&.call(stream)
+										if body.stream?
+											return body.call(write_tunnel_body(version))
+										else
+											write_tunnel_body(version, body)
+										end
 									else
 										write_response(@version, response.status, response.headers)
 										
 										if request.connect? and response.success?
-											stream = write_tunnel_body(version)
-											
 											# Same as above:
 											request = nil
 											response = nil
 											
-											# We must return here as no further request processing can be done:
-											return body.call(stream)
+											if body.stream?
+												return body.call(write_tunnel_body(version))
+											else
+												write_tunnel_body(version, body)
+											end
 										else
 											head = request.head?
 											
@@ -143,8 +144,12 @@ module Async
 									request&.finish
 								end
 								
-								# Discard or wait for the input body to be consumed:
-								finishable&.wait
+								if finishable
+									finishable.wait(@persistent)
+								else
+									# Do not remove this line or you will unleash the gods of concurrency hell.
+									task.yield
+								end
 							rescue => error
 								raise
 							ensure

data/lib/async/http/protocol/http2/output.rb

@@ -17,7 +17,8 @@ module Async
 						
 						@task = nil
 						
-						@window_updated = Async::Condition.new
+						@guard = ::Mutex.new
+						@window_updated = ::ConditionVariable.new
 					end
 					
 					attr :trailer
@@ -33,17 +34,26 @@ module Async
 					end
 					
 					def window_updated(size)
-						@window_updated.signal
+						@guard.synchronize do
+							@window_updated.signal
+						end
 					end
 					
 					def write(chunk)
 						until chunk.empty?
 							maximum_size = @stream.available_frame_size
 							
-							while maximum_size <= 0
-								@window_updated.wait
-								
-								maximum_size = @stream.available_frame_size
+							# We try to avoid synchronization if possible:
+							if maximum_size <= 0
+								@guard.synchronize do
+									maximum_size = @stream.available_frame_size
+									
+									while maximum_size <= 0
+										@window_updated.wait(@guard)
+										
+										maximum_size = @stream.available_frame_size
+									end
+								end
 							end
 							
 							break unless chunk = send_data(chunk, maximum_size)

data/lib/async/http/protocol/http2/stream.rb

@@ -62,9 +62,9 @@ module Async
 						end
 						
 						# TODO this might need to be in an ensure block:
-						if @input and frame.end_stream?
-							@input.close_write
+						if input = @input and frame.end_stream?
 							@input = nil
+							input.close_write
 						end
 					rescue ::Protocol::HTTP2::HeaderError => error
 						Console.logger.debug(self, error)
@@ -123,6 +123,8 @@ module Async
 					
 					# Called when the output terminates normally.
 					def finish_output(error = nil)
+						return if self.closed?
+						
 						trailer = @output&.trailer
 						
 						@output = nil
@@ -152,14 +154,14 @@ module Async
 					def closed(error)
 						super
 						
-						if @input
-							@input.close_write(error)
+						if input = @input
 							@input = nil
+							input.close_write(error)
 						end
 						
-						if @output
-							@output.stop(error)
+						if output = @output
 							@output = nil
+							output.stop(error)
 						end
 						
 						if pool = @pool and @connection

data/lib/async/http/version.rb

@@ -5,6 +5,6 @@
 
 module Async
 	module HTTP
-		VERSION = "0.78.0"
+		VERSION = "0.80.0"
 	end
 end

data.tar.gz.sig

@@ -1,4 +1 @@
-M7>d[܌c��a���0�d�}�bB�զ�UƐ��(�)��e���
-^�Ƶ=U���Z��'
�cOt3JlLO�K6�4�1:$�t��6�ݚ���)�[y��q��^�a7��.x쥃��U�ɀ�N�*5�T�Y������
-h%!5��c;Z[j�sX�L�����}>m}7e��p|9"̕$�4������k'Me_�{s1�?8/Li�Na���-ZI�4���1��G�$a]��ŋ��*���~-N�D8��&�
����"��pr:���P;2��:a[��UB��	)|�}4[��ق�`����lq
-��8=�5���Si/�ﻲH���^	�fO
+ug;oFa�l��C�7vt}}�5�X�n��l�Q�6�^h��:tQ[���3d9%���@:B�/�1�����0�K�J�� �LU����o'y�	O�{��.醠K\/���|�k<{p*P��)X��"k���1�c��`~!�V�7=��m�){3BT>���0��6+B�B�Q�w�q3��'ˇ%�.nYZ8�7��q�����u�]���`C4<C��ѿߨ!_���=YE�!�w��DŽ��ǒ�:X8�SL+�59����b��Ձ���,����2D?F���|\;�S����?
����q��\������Ӝ2n���rP�7YVUԠ,z���xs��

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: async-http
 version: !ruby/object:Gem::Version
-  version: 0.78.0
+  version: 0.80.0
 platform: ruby
 authors:
 - Samuel Williams
@@ -58,7 +58,7 @@ cert_chain:
   Q2K9NVun/S785AP05vKkXZEFYxqG6EW012U4oLcFl5MySFajYXRYbuUpH6AY+HP8
   voD0MPg1DssDLKwXyt1eKD/+Fq0bFWhwVM/1XiAXL7lyYUyOq24KHgQ2Csg=
   -----END CERTIFICATE-----
-date: 2024-09-22 00:00:00.000000000 Z
+date: 2024-10-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: async
@@ -136,28 +136,28 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.25'
+        version: '0.27'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.25'
+        version: '0.27'
 - !ruby/object:Gem::Dependency
   name: protocol-http2
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.18'
+        version: '0.19'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.18'
+        version: '0.19'
 - !ruby/object:Gem::Dependency
   name: traces
   requirement: !ruby/object:Gem::Requirement