async-http 0.78.0 → 0.79.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 76e78af8d20808d557b1c13c850af0a268de5535726ba07708b2012eb8752029
-  data.tar.gz: 53bdf9b950394a03ca8c1fd47befe51e8ba51a5cce82aa47c01111df7efd21b4
+  metadata.gz: 93d3d9d22a84537ba522b417586a7bc02d243d56defe26fb9a3937470675b244
+  data.tar.gz: 8c16add5b2c301948235c739d076f92c400cdd745cc36881fac49125a41c5bc4
 SHA512:
-  metadata.gz: 43c49b50bb88955088b74926e3fe33310f38f4b09d9b9ecdc538a9835c9be5b8be0e172e09794324a53db99e09b0102ae96fc179af13578a758f42644d6f4011
-  data.tar.gz: fa1a0d97f326bdd72010f07cfe21004b0da8f75fe4556778bd3b918f42e2e85f22bc41cd07e2a64b4395540e531a879480e634a8fe401ee0d0d7c146af8b61d7
+  metadata.gz: cf362c7394967d2746e6d37984f25f5fc02e8e1fa89b7d5f6afcf31d87e665d75877cd5a9cd920f1ae87b6ecbc73e7d9b7cedc72467a69d95b5d24a04b897a8e
+  data.tar.gz: e922cf9cfd745423d9350f633d1a152dd8f6e06ae23abd3c8b1e403f49abb7089181aa68a1ded1a329cf2017b7bf98feb6a60fed45e1f8582eb3b01ca401ce4d

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/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.79.0"
 	end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: async-http
 version: !ruby/object:Gem::Version
-  version: 0.78.0
+  version: 0.79.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-09-24 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