async-http 0.98.1 → 0.100.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 32a7056790328efb71a89aa3c721672b18640bef1ac9c6ba67ce934f085f3c64
-  data.tar.gz: df9f6b4843347757a887d9ab64e49af66e2e88cfc6dcc8659d77dc875f0d02b5
+  metadata.gz: ee704ca93c1b809cfcd2ea2db7961066a10d433143f1144658a52c7674179110
+  data.tar.gz: 268a538e9a234f51495990e39d8f5c9f7bb8d1a73db57da285e6cf180f3638e7
 SHA512:
-  metadata.gz: 040b5ef80794ddda14b86cdfcb118fd63de4ee72c21c8923e740b2ad3cfb7138370edf87c0db5249b6e177009ba4c7683cc6ab8515e3f94720ab74462839e669
-  data.tar.gz: 2f5f60d796f3fd4d1dcad6f8dfd9acd11ed40c82a3092f65cdf22093e9c2019a558cb91085978a76a78be78596207bebfa7a2554fd8eac0c2503f268829cd999
+  metadata.gz: 9393381596b9b8f1c14cea1383577cb4037e60f74d4e0aa9f84862f38f906988645fc9153dc68cb933fe3932ea3b3e5abc92cae9429200d4ee1a17a90706b476
+  data.tar.gz: 9bae0d3084ce01b65c9eeb208b97a525b176063c20473f554a1b18016b486e6a0813b974864c7bbdadaca22e1605932e1239e10c402bf38a9cfd16fb59744c2e

data/lib/async/http/client.rb

@@ -9,6 +9,7 @@ require "io/endpoint"
 require "async/pool/controller"
 
 require "protocol/http/body/completable"
+require "protocol/http/error"
 require "protocol/http/methods"
 
 require_relative "protocol"
@@ -126,7 +127,7 @@ module Async
 					else
 						raise
 					end
-				rescue SocketError, IOError, EOFError, Errno::ECONNRESET, Errno::EPIPE
+				rescue ::Protocol::HTTP::RemoteError, SocketError, IOError, EOFError, Errno::ECONNRESET, Errno::EPIPE
 					if connection
 						@pool.release(connection)
 						connection = nil

data/lib/async/http/endpoint.rb

@@ -70,6 +70,7 @@ module Async
 			# @option hostname [String] the hostname to connect to (or bind to), overrides the URL hostname (used for SNI).
 			# @option port [Integer] the port to bind to, overrides the URL port.
 			# @option ssl_context [OpenSSL::SSL::SSLContext] the context to use for TLS.
+			# @option tls_configuration [IO::Endpoint::TLS::Configuration] the transport-neutral TLS configuration.
 			# @option alpn_protocols [Array(String)] the alpn protocols to negotiate.
 			def initialize(url, endpoint = nil, **options)
 				super(**options)
@@ -206,6 +207,11 @@ module Async
 				end
 			end
 			
+			# @returns [IO::Endpoint::TLS::Configuration | Nil] The transport-neutral TLS configuration.
+			def tls_configuration
+				@options[:tls_configuration]
+			end
+			
 			# Build a suitable endpoint, optionally wrapping in TLS for secure connections.
 			# @parameter endpoint [IO::Endpoint::Generic | Nil] An optional underlying endpoint to wrap.
 			# @returns [IO::Endpoint::Generic] The constructed endpoint.
@@ -216,6 +222,7 @@ module Async
 					# Wrap it in SSL:
 					return ::IO::Endpoint::SSLEndpoint.new(endpoint,
 						ssl_context: self.ssl_context,
+						tls_configuration: self.tls_configuration,
 						hostname: @url.hostname,
 						timeout: self.timeout,
 					)
@@ -280,6 +287,7 @@ module Async
 				options.delete(:port)
 				options.delete(:hostname)
 				options.delete(:ssl_context)
+				options.delete(:tls_configuration)
 				options.delete(:alpn_protocols)
 				options.delete(:protocol)
 				

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

@@ -10,6 +10,7 @@
 require_relative "connection"
 require_relative "finishable"
 
+require "async/promise"
 require "console/event/failure"
 
 module Async
@@ -22,14 +23,14 @@ module Async
 					def initialize(...)
 						super
 						
-						@ready = Async::Notification.new
+						@ready = Async::Promise.new
 					end
 					
 					# Called when the connection is closed, signalling any waiting tasks.
 					def closed(error = nil)
 						super
 						
-						@ready.signal
+						@ready.resolve(nil)
 					end
 					
 					# Write a failure response with the given status code.

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

@@ -6,6 +6,8 @@
 require_relative "../response"
 require_relative "stream"
 
+require "async/promise"
+
 module Async
 	module HTTP
 		module Protocol
@@ -20,8 +22,7 @@ module Async
 							
 							@response = Response.new(self)
 							
-							@notification = Async::Notification.new
-							@exception = nil
+							@ready = Async::Promise.new
 						end
 						
 						attr :response
@@ -88,7 +89,7 @@ module Async
 								send_reset_stream(::Protocol::HTTP2::Error::PROTOCOL_ERROR)
 							end
 							
-							self.notify!
+							@ready.resolve(nil)
 							
 							return headers
 						end
@@ -106,22 +107,15 @@ module Async
 							@response.request.send_interim_response(status, headers)
 						end
 						
-						# Notify anyone waiting on the response headers to be received (or failure).
-						def notify!
-							if notification = @notification
-								@notification = nil
-								notification.signal
-							end
-						end
-						
 						# Wait for the headers to be received or for stream reset.
 						def wait
-							# If you call wait after the headers were already received, it should return immediately:
-							@notification&.wait
-							
-							if @exception
-								raise @exception
+							@ready.wait
+						rescue ::Protocol::HTTP2::StreamError => error
+							if error.code == ::Protocol::HTTP2::Error::INTERNAL_ERROR
+								raise ::Protocol::HTTP::RemoteError, error.message
 							end
+							
+							raise
 						end
 						
 						# Called when the stream is closed.
@@ -133,9 +127,11 @@ module Async
 								@response = nil
 							end
 							
-							@exception = error
-							
-							self.notify!
+							if error
+								@ready.reject(error)
+							else
+								@ready.resolve(nil)
+							end
 						end
 					end
 					

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

@@ -6,6 +6,7 @@
 require_relative "connection"
 require_relative "request"
 
+require "async/barrier"
 require "protocol/http2/server"
 
 module Async
@@ -41,11 +42,7 @@ module Async
 					
 					# Close the server connection and stop accepting requests.
 					def close(error = nil)
-						if @requests
-							# Stop the request loop:
-							@requests.enqueue(nil)
-							@requests = nil
-						end
+						@requests.close
 						
 						super
 					end
@@ -55,9 +52,10 @@ module Async
 					# 	@parameter request [Request] The incoming HTTP/2 request.
 					def each(task: Task.current)
 						task.annotate("Reading #{version} requests for #{self.class}.")
+						barrier = Async::Barrier.new(parent: task)
 						
 						# It's possible the connection has died before we get here...
-						@requests&.async do |task, request|
+						@requests.async(parent: barrier) do |task, request|
 							task.annotate("Incoming request: #{request.method} #{request.path.inspect}.")
 							
 							response = nil
@@ -79,7 +77,9 @@ module Async
 							end
 						end
 						
-						# Maybe we should add some synchronisation here - i.e. only exit once all requests are finished.
+						barrier.wait
+					ensure
+						barrier&.stop
 					end
 				end
 			end

data/lib/async/http/version.rb

@@ -7,6 +7,6 @@
 module Async
 	# @namespace
 	module HTTP
-		VERSION = "0.98.1"
+		VERSION = "0.100.0"
 	end
 end

data/readme.md

@@ -16,6 +16,14 @@ Please see the [project documentation](https://socketry.github.io/async-http/) f
 
 Please see the [project releases](https://socketry.github.io/async-http/releases/index) for all releases.
 
+### v0.100.0
+
+  - Added transport-neutral TLS configuration support to `Async::HTTP::Endpoint`.
+
+### v0.99.0
+
+  - Retry safe requests when a remote HTTP/2 endpoint resets the stream with `INTERNAL_ERROR` before returning a response.
+
 ### v0.98.1
 
   - Probe idle HTTP/1 connections before reuse, avoiding requests on connections already closed by the peer.
@@ -51,14 +59,6 @@ Please see the [project releases](https://socketry.github.io/async-http/releases
 
   - Fix `defer_stop` usage in `HTTP1::Server`, improving server graceful shutdown behavior.
 
-### v0.94.0
-
-  - Propagate all errors from background reader to active streams so that they are closed correctly (e.g. errors are not missed).
-
-### v0.92.2
-
-  - Better handling of trailers. If invalid trailers are received, the connection (HTTP/1) or stream (HTTP/2) is reset.
-
 ## See Also
 
   - [benchmark-http](https://github.com/socketry/benchmark-http) — A benchmarking tool to report on web server concurrency.

data/releases.md

@@ -1,5 +1,13 @@
 # Releases
 
+## v0.100.0
+
+  - Added transport-neutral TLS configuration support to `Async::HTTP::Endpoint`.
+
+## v0.99.0
+
+  - Retry safe requests when a remote HTTP/2 endpoint resets the stream with `INTERNAL_ERROR` before returning a response.
+
 ## v0.98.1
 
   - Probe idle HTTP/1 connections before reuse, avoiding requests on connections already closed by the peer.

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: async-http
 version: !ruby/object:Gem::Version
-  version: 0.98.1
+  version: 0.100.0
 platform: ruby
 authors:
 - Samuel Williams
@@ -96,14 +96,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.14'
+        version: '0.18'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.14'
+        version: '0.18'
 - !ruby/object:Gem::Dependency
   name: io-stream
   requirement: !ruby/object:Gem::Requirement
@@ -124,14 +124,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.64'
+        version: '0.66'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.64'
+        version: '0.66'
 - !ruby/object:Gem::Dependency
   name: protocol-http1
   requirement: !ruby/object:Gem::Requirement