async-http 0.82.0 → 0.82.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 78fb5808dae4ef161e8ae21a72c9ec06dd740eced5e7afcc57b7de091f728ffa
-  data.tar.gz: 0d9b421eb5f7130983b1d5903da05b113718fdfcaef497eeace3dae0a57c98d1
+  metadata.gz: ba3cc6cc766d5f45b1480cfaa887ffc1871b7929b679ad49185eb81cfc3f1eb8
+  data.tar.gz: 230f4aee8068ed5fe0a2059111f462eeda6278fdd3354c092c5631487a96daa3
 SHA512:
-  metadata.gz: e807158cf95b6f822bc84d8592c296c2bff6700985372c54425a679385383ab23c143cd0e615bf9f3a5b0450a602ce3862cfa3bbdf93426cd680993b94d6a74d
-  data.tar.gz: 2585ec5a381fc5c2411ed6205cae1879534aff66c250b50f61acf4d9d4a0c50fdaf8a997baa29191b32effdde185f5e4b33101d99095f8ac8a51e3fae44f0c33
+  metadata.gz: 4b754907b1cf5907481470edb66a6f0d9528ab52f5cddddffdaf7cf61fd17cb6ae8a01629d80e54bfd02ab0469cbc8a14073131be9b0da3f2b443d94ff00e476
+  data.tar.gz: c9a55134034af4597e19b317ef42f62f5edc377d5b8a90c1de19a3c3f713ed2c120a939c4cd84c3758de0935e1e6b65a28347db228ed2393925c773bdfff258b

data/lib/async/http/client.rb

@@ -101,7 +101,7 @@ module Async
 					# As we cache pool, it's possible these pool go bad (e.g. closed by remote host). In this case, we need to try again. It's up to the caller to impose a timeout on this. If this is the last attempt, we force a new connection.
 					connection = @pool.acquire
 					
-					response = make_response(request, connection)
+					response = make_response(request, connection, attempt)
 					
 					# This signals that the ensure block below should not try to release the connection, because it's bound into the response which will be returned:
 					connection = nil
@@ -140,6 +140,36 @@ module Async
 				"#<#{self.class} authority=#{@authority.inspect}>"
 			end
 			
+			protected
+			
+			def make_response(request, connection, attempt)
+				response = request.call(connection)
+				
+				response.pool = @pool
+				
+				return response
+			end
+			
+			def assign_default_tags(tags)
+				tags[:endpoint] = @endpoint.to_s
+				tags[:protocol] = @protocol.to_s
+			end
+			
+			def make_pool(**options)
+				if connection_limit = options.delete(:connection_limit)
+					warn "The connection_limit: option is deprecated, please use limit: instead.", uplevel: 2
+					options[:limit] = connection_limit
+				end
+				
+				self.assign_default_tags(options[:tags] ||= {})
+				
+				Async::Pool::Controller.wrap(**options) do
+					Console.logger.debug(self) {"Making connection to #{@endpoint.inspect}"}
+					
+					@protocol.client(@endpoint.connect)
+				end
+			end
+			
 			Traces::Provider(self) do
 				def call(request)
 					attributes = {
@@ -178,35 +208,15 @@ module Async
 						end
 					end
 				end
-			end
-			
-			protected
-			
-			def make_response(request, connection)
-				response = request.call(connection)
-				
-				response.pool = @pool
 				
-				return response
-			end
-			
-			def assign_default_tags(tags)
-				tags[:endpoint] = @endpoint.to_s
-				tags[:protocol] = @protocol.to_s
-			end
-			
-			def make_pool(**options)
-				if connection_limit = options.delete(:connection_limit)
-					warn "The connection_limit: option is deprecated, please use limit: instead.", uplevel: 2
-					options[:limit] = connection_limit
-				end
-				
-				self.assign_default_tags(options[:tags] ||= {})
-				
-				Async::Pool::Controller.wrap(**options) do
-					Console.logger.debug(self) {"Making connection to #{@endpoint.inspect}"}
+				def make_response(request, connection, attempt)
+					attributes = {
+						attempt: attempt,
+					}
 					
-					@protocol.client(@endpoint.connect)
+					Traces.trace("async.http.client.make_response", attributes: attributes) do
+						super
+					end
 				end
 			end
 		end

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

@@ -61,10 +61,8 @@ module Async
 							self.receive_initial_headers(super, frame.end_stream?)
 						end
 						
-						# TODO this might need to be in an ensure block:
-						if input = @input and frame.end_stream?
-							@input = nil
-							input.close_write
+						if @input and frame.end_stream?
+							@input.close_write
 						end
 					rescue ::Protocol::HTTP2::HeaderError => error
 						Console.logger.debug(self, error)
@@ -103,7 +101,6 @@ module Async
 							
 							if frame.end_stream?
 								@input.close_write
-								@input = nil
 							end
 						end
 						

data/lib/async/http/version.rb

@@ -5,6 +5,6 @@
 
 module Async
 	module HTTP
-		VERSION = "0.82.0"
+		VERSION = "0.82.2"
 	end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: async-http
 version: !ruby/object:Gem::Version
-  version: 0.82.0
+  version: 0.82.2
 platform: ruby
 authors:
 - Samuel Williams
@@ -58,7 +58,7 @@ cert_chain:
   Q2K9NVun/S785AP05vKkXZEFYxqG6EW012U4oLcFl5MySFajYXRYbuUpH6AY+HP8
   voD0MPg1DssDLKwXyt1eKD/+Fq0bFWhwVM/1XiAXL7lyYUyOq24KHgQ2Csg=
   -----END CERTIFICATE-----
-date: 2024-10-15 00:00:00.000000000 Z
+date: 2024-10-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: async