async-http 0.56.1 → 0.56.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7d7ce0d0d3e46cbefd67fdd2064c6a144b8dfda461363462e05ea2bc421a1150
-  data.tar.gz: '0075078bcb61b68205d0094047df19f38a469b8b545811336a35e3c160bb5880'
+  metadata.gz: 7152de875b7dee12eed4876f7a8cd55ce3c091011d87fdc9389fb52bae092686
+  data.tar.gz: 812fa1a3eb74072dac8d10fa074c6f442a9017e2f9d65b2f53fe9fd72db0d90d
 SHA512:
-  metadata.gz: '061618238094db458acb14c1ce6273af279f0571d5c67b1b49a4cc03d12228c4269e45be544b69edb555a9d304f22564306193bd9205262853285940dd7ccf8b'
-  data.tar.gz: 63e7b3767a42cc2fc82e81a55bfb5e3aa64ea16d83226a3452b9453266f9bbff1cbef9ea015958d77c89d142ff38f68c8c2b36216df80ad1c7320e868fd0beba
+  metadata.gz: 9e3d63e16368697ad7e146b3448cf5b19ae7147db9f65356252dcee05a83fb852e2a03350bbe591eab21a530100d1c3830b9d76758f81e21366c45542a8a5bbb
+  data.tar.gz: b9d013071286319e2a45ca85de0dad52fe030f1230319fb68d7ec5f4d1d999e96fbbba23cebf0021ff32c826c597d16c862c834f98d3bb2773f13fb6a56f242d

data/bake/async/http/h2spec.rb

@@ -26,7 +26,7 @@ def server
 	
 	container = Async::Container.new
 	
-	Async.logger.info(self){"Starting server..."}
+	Console.logger.info(self){"Starting server..."}
 	
 	container.run(count: 1) do
 		server = Async::HTTP::Server.for(endpoint, protocol: Async::HTTP::Protocol::HTTP2, scheme: "https") do |request|

data/lib/async/http/client.rb

@@ -83,7 +83,7 @@ module Async
 			
 			def close
 				while @pool.busy?
-					Async.logger.warn(self) {"Waiting for #{@protocol} pool to drain: #{@pool}"}
+					Console.logger.warn(self) {"Waiting for #{@protocol} pool to drain: #{@pool}"}
 					@pool.wait
 				end
 				
@@ -152,7 +152,7 @@ module Async
 			
 			def make_pool(connection_limit)
 				Async::Pool::Controller.wrap(limit: connection_limit) do
-					Async.logger.debug(self) {"Making connection to #{@endpoint.inspect}"}
+					Console.logger.debug(self) {"Making connection to #{@endpoint.inspect}"}
 					
 					@protocol.client(@endpoint.connect)
 				end

data/lib/async/http/internet.rb

@@ -31,7 +31,7 @@ module Async
 	module HTTP
 		class Internet
 			def initialize(**options)
-				@clients = {}
+				@clients = Hash.new
 				@options = options
 			end
 			
@@ -39,13 +39,17 @@ module Async
 			# @attribute [Hash(URI, Client)]
 			attr :clients
 			
-			def call(method, url, headers = nil, body = nil)
-				endpoint = Endpoint.parse(url)
+			def client_for(endpoint)
 				key = host_key(endpoint)
 				
-				client = @clients.fetch(key) do
-					@clients[key] = self.client_for(endpoint)
+				@clients.fetch(key) do
+					@clients[key] = self.make_client(endpoint)
 				end
+			end
+			
+			def call(method, url, headers = nil, body = nil)
+				endpoint = Endpoint.parse(url)
+				client = self.client_for(endpoint)
 				
 				body = Body::Buffered.wrap(body)
 				headers = ::Protocol::HTTP::Headers[headers]
@@ -55,12 +59,6 @@ module Async
 				return client.call(request)
 			end
 			
-			def client_for(endpoint)
-				::Protocol::HTTP::AcceptEncoding.new(
-					Client.new(endpoint, **@options)
-				)
-			end
-			
 			def close
 				# The order of operations here is to avoid a race condition between iterating over clients (#close may yield) and creating new clients.
 				clients = @clients.values
@@ -75,7 +73,13 @@ module Async
 				end
 			end
 			
-			private
+			protected
+			
+			def make_client(endpoint)
+				::Protocol::HTTP::AcceptEncoding.new(
+					Client.new(endpoint, **@options)
+				)
+			end
 			
 			def host_key(endpoint)
 				url = endpoint.url.dup

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

@@ -29,7 +29,10 @@ module Async
 				class Client < Connection
 					# Used by the client to send requests to the remote server.
 					def call(request, task: Task.current)
-						Async.logger.debug(self) {"#{request.method} #{request.path} #{request.headers.inspect}"}
+						# We need to keep track of connections which are not in the initial "ready" state.
+						@ready = false
+						
+						Console.logger.debug(self) {"#{request.method} #{request.path} #{request.headers.inspect}"}
 						
 						trailer = request.headers.trailer!
 						
@@ -72,7 +75,10 @@ module Async
 							write_body(@version, body, false, trailer)
 						end
 						
-						return Response.read(self, request)
+						response = Response.read(self, request)
+						@ready = true
+						
+						return response
 					rescue
 						# This will ensure that #reusable? returns false.
 						@stream.close

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

@@ -33,6 +33,7 @@ module Async
 					def initialize(stream, version)
 						super(stream)
 						
+						@ready = true
 						@version = version
 					end
 					
@@ -48,8 +49,6 @@ module Async
 					
 					def read_line?
 						@stream.read_until(CRLF)
-					rescue Errno::ECONNRESET
-						return nil
 					end
 					
 					def read_line
@@ -68,11 +67,11 @@ module Async
 					
 					# Can we use this connection to make requests?
 					def viable?
-						@stream&.connected?
+						@ready && @stream&.connected?
 					end
 					
 					def reusable?
-						@persistent && @stream && !@stream.closed?
+						@ready && @persistent && @stream && !@stream.closed?
 					end
 				end
 			end

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

@@ -88,7 +88,7 @@ module Async
 							@input = nil
 						end
 					rescue ::Protocol::HTTP2::HeaderError => error
-						Async.logger.error(self, error)
+						Console.logger.error(self, error)
 						
 						send_reset_stream(error.code)
 					end

data/lib/async/http/protocol/https.rb

@@ -59,7 +59,7 @@ module Async
 					# alpn_protocol is only available if openssl v1.0.2+
 					name = peer.alpn_protocol
 					
-					Async.logger.debug(self) {"Negotiating protocol #{name.inspect}..."}
+					Console.logger.debug(self) {"Negotiating protocol #{name.inspect}..."}
 					
 					if protocol = HANDLERS[name]
 						return protocol

data/lib/async/http/server.rb

@@ -48,7 +48,7 @@ module Async
 			def accept(peer, address, task: Task.current)
 				connection = @protocol.server(peer)
 				
-				Async.logger.debug(self) {"Incoming connnection from #{address.inspect} to #{@protocol}"}
+				Console.logger.debug(self) {"Incoming connnection from #{address.inspect} to #{@protocol}"}
 				
 				connection.each do |request|
 					# We set the default scheme unless it was otherwise specified.
@@ -58,7 +58,7 @@ module Async
 					# This is a slight optimization to avoid having to get the address from the socket.
 					request.remote_address = address
 					
-					# Async.logger.debug(self) {"Incoming request from #{address.inspect}: #{request.method} #{request.path}"}
+					# Console.logger.debug(self) {"Incoming request from #{address.inspect}: #{request.method} #{request.path}"}
 					
 					# If this returns nil, we assume that the connection has been hijacked.
 					self.call(request)

data/lib/async/http/version.rb

@@ -22,6 +22,6 @@
 
 module Async
 	module HTTP
-		VERSION = "0.56.1"
+		VERSION = "0.56.5"
 	end
 end

metadata

@@ -1,55 +1,55 @@
 --- !ruby/object:Gem::Specification
 name: async-http
 version: !ruby/object:Gem::Version
-  version: 0.56.1
+  version: 0.56.5
 platform: ruby
 authors:
 - Samuel Williams
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-04-23 00:00:00.000000000 Z
+date: 2021-07-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: async
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '1.25'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '1.25'
 - !ruby/object:Gem::Dependency
   name: async-io
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '1.28'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '1.28'
 - !ruby/object:Gem::Dependency
   name: async-pool
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0.2'
 - !ruby/object:Gem::Dependency
@@ -244,7 +244,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.2
+rubygems_version: 3.2.22
 signing_key:
 specification_version: 4
 summary: A HTTP client and server library.