async-http 0.82.1 → 0.82.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3a11ccce0d608ca64fbed5c7e5cc23428e25d618f0b6b68aa34e6be7516a6982
-  data.tar.gz: 890277e43541eaacb5f9f756f697505bcf8443e5aeea7d99b43d23df94896f87
+  metadata.gz: 34b971dd39f2aeab82cdca2bdb30c6d51a8c05b920933e5daf8e5525a60b2362
+  data.tar.gz: 53d7cfbb2541b3d138198d2ab976e59b10709fb048c3cdaf9e371f21aeb8ae57
 SHA512:
-  metadata.gz: 2748cb15c2ae647b7aad72c3151edf747f4c1a366c495c981a0a135d1219b4bccdf5f8433ca44e79d1e92f9e3d4e54a775747df97a0ecede05982e6451f538fb
-  data.tar.gz: 0647bec2b9b92d5fddbf9ef43bdaa53d626026172d5dc0a922da0716a0c3cde75e8d586a12f79ad851f98994c30c6a1b534e78dd96bd83ee061778ee243cfb41
+  metadata.gz: 20bdd7ab4ebf36833e1e05542dbe8288534fd37963a7ad25552b35a2dc8cb1dd08729eaa38dda1bcec084e4534105231a5901af99cf0f28f4a65621315bcfd5a
+  data.tar.gz: 8e6d0f0e569ad1f37f50265231f2056099eacdd8f9a7a17bffff3f894f5fa50531a32a483ca11d5ff5d5d50c47cfbea4e31825319e4b7438a026df21cdb9e4d8

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/http1/client.rb

@@ -5,6 +5,8 @@
 
 require_relative "connection"
 
+require "traces/provider"
+
 module Async
 	module HTTP
 		module Protocol
@@ -79,6 +81,20 @@ module Async
 						self.close(error)
 						raise
 					end
+					
+					Traces::Provider(self) do
+						def write_request(...)
+							Traces.trace("async.http.protocol.http1.client.write_request") do
+								super
+							end
+						end
+						
+						def read_response(...)
+							Traces.trace("async.http.protocol.http1.client.read_response") do
+								super
+							end
+						end
+					end
 				end
 			end
 		end

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

@@ -6,6 +6,7 @@
 require_relative "connection"
 require_relative "response"
 
+require "traces/provider"
 require "protocol/http2/client"
 
 module Async
@@ -34,10 +35,32 @@ module Async
 						@count += 1
 						
 						response = create_response
+						write_request(response, request)
+						read_response(response)
+						
+						return response
+					end
+					
+					def write_request(response, request)
 						response.send_request(request)
+					end
+					
+					def read_response(response)
 						response.wait
+					end
+					
+					Traces::Provider(self) do
+						def write_request(...)
+							Traces.trace("async.http.protocol.http2.client.write_request") do
+								super
+							end
+						end
 						
-						return response
+						def read_response(...)
+							Traces.trace("async.http.protocol.http2.client.read_response") do
+								super
+							end
+						end
 					end
 				end
 			end

data/lib/async/http/version.rb

@@ -5,6 +5,6 @@
 
 module Async
 	module HTTP
-		VERSION = "0.82.1"
+		VERSION = "0.82.3"
 	end
 end

metadata

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