async-http 0.72.0 → 0.73.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8b358f54acd8cf0943e35f30865d7f505025bad37a2c37e0dda5f8438efe3b25
-  data.tar.gz: a8a9eee82cc0723cf8101ef727072d4518a38ddb7e68f5f0f2d15027e92e83a9
+  metadata.gz: 01cf4d909911a28f4f386982504e5865d9053210af1cb121a4acc8e0812a63bd
+  data.tar.gz: 20b913990be812792c72f5e68649ee8084310112287e316c438c6a220cee9982
 SHA512:
-  metadata.gz: 7f5720d8ac42c48afb19bd5c387db2da015b2d210836c6d1a7abfe3cd3a4df85587c92b9bbdee61fce397302c66c8ca303c123d9e7f9a2201b26602ea0240e58
-  data.tar.gz: 91a543963515d1375953da096781b7b9d5a161e20bafc6aa067fbf0e09ff546ae3b48e18a5d8ca06dc5173184546898d415310b6bfbded5605040bce08443010
+  metadata.gz: 6b1eefc4c6f0d1a631c1b6f0365ae36a7fe4067e1b372f334473b99f379f0755d29d53a5a04183f5eef46a9c0cf6504a2a56b7c5741757700b44d5c2283fb129
+  data.tar.gz: 1ed32dc7b17b30ddd2d842e688e89df28a0cc51a0610398ea567191c6158c5adde2ebdd337035450fca499b0c312e2d9b3bed616f9cf1058c1d136bbdd10acd4

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

@@ -24,7 +24,7 @@ module Async
 						# HTTP/1 requests with an upgrade header (which can contain zero or more values) are extracted into the protocol field of the request, and we expect a response to select one of those protocols with a status code of 101 Switching Protocols.
 						protocol = headers.delete('upgrade')
 						
-						super(nil, authority, method, path, version, headers, body, protocol)
+						super(nil, authority, method, path, version, headers, body, protocol, self.public_method(:write_interim_response))
 					end
 					
 					def connection
@@ -39,8 +39,8 @@ module Async
 						@connection.hijack!
 					end
 					
-					def write_interim_response(response)
-						@connection.write_interim_response(response.version, response.status, response.headers)
+					def write_interim_response(status, headers = nil)
+						@connection.write_interim_response(@version, status, headers)
 					end
 				end
 			end

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

@@ -17,6 +17,8 @@ module Async
 							
 							if response.final?
 								return response
+							else
+								request.send_interim_response(response.status, response.headers)
 							end
 						end
 					end

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

@@ -23,6 +23,8 @@ module Async
 						attr :request
 						
 						def receive_initial_headers(headers, end_stream)
+							@headers = ::Protocol::HTTP::Headers.new
+							
 							headers.each do |key, value|
 								if key == SCHEME
 									raise ::Protocol::HTTP2::HeaderError, "Request scheme already specified!" if @request.scheme
@@ -85,7 +87,7 @@ module Async
 					end
 					
 					def initialize(stream)
-						super(nil, nil, nil, nil, VERSION, nil)
+						super(nil, nil, nil, nil, VERSION, nil, nil, nil, self.public_method(:write_interim_response))
 						
 						@stream = stream
 					end
@@ -117,10 +119,6 @@ module Async
 							[STATUS, response.status],
 						]
 						
-						if protocol = response.protocol
-							protocol_headers << [PROTOCOL, protocol]
-						end
-						
 						if length = response.body&.length
 							protocol_headers << [CONTENT_LENGTH, length]
 						end
@@ -142,14 +140,16 @@ module Async
 						end
 					end
 					
-					def write_interim_response(response)
-						protocol_headers = [
-							[STATUS, response.status]
+					def write_interim_response(status, headers = nil)
+						interim_response_headers = [
+							[STATUS, status]
 						]
 						
-						headers = ::Protocol::HTTP::Headers::Merged.new(protocol_headers, response.headers)
+						if headers
+							interim_response_headers = ::Protocol::HTTP::Headers::Merged.new(interim_response_headers, headers)
+						end
 						
-						@stream.send_headers(nil, headers)
+						@stream.send_headers(nil, interim_response_headers)
 					end
 				end
 			end

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

@@ -38,18 +38,25 @@ module Async
 						
 						# This should be invoked from the background reader, and notifies the task waiting for the headers that we are done.
 						def receive_initial_headers(headers, end_stream)
+							# While in theory, the response pseudo-headers may be extended in the future, currently they only response pseudo-header is :status, so we can assume it is always the first header.
+							status_header = headers.shift
+							
+							if status_header.first != ':status'
+								raise ProtocolError, "Invalid response headers: #{headers.inspect}"
+							end
+							
+							status = Integer(status_header.last)
+							
+							if status >= 100 && status < 200
+								return receive_interim_headers(status, headers)
+							end
+							
+							@response.status = status
+							@headers = ::Protocol::HTTP::Headers.new
+							
 							headers.each do |key, value|
 								# It's guaranteed that this should be the first header:
-								if key == STATUS
-									status = Integer(value)
-									
-									# Ignore informational headers:
-									return if status >= 100 && status < 200
-									
-									@response.status = Integer(value)
-								elsif key == PROTOCOL
-									@response.protocol = value
-								elsif key == CONTENT_LENGTH
+								if key == CONTENT_LENGTH
 									@length = Integer(value)
 								else
 									add_header(key, value)
@@ -74,6 +81,16 @@ module Async
 							return headers
 						end
 						
+						def receive_interim_headers(status, headers)
+							if headers.any?
+								headers = ::Protocol::HTTP::Headers[headers]
+							else
+								headers = nil
+							end
+							
+							@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

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

@@ -51,10 +51,9 @@ module Async
 					end
 					
 					def process_headers(frame)
-						if frame.end_stream? && @headers
+						if @headers and frame.end_stream?
 							self.receive_trailing_headers(super, frame.end_stream?)
 						else
-							@headers ||= ::Protocol::HTTP::Headers.new
 							self.receive_initial_headers(super, frame.end_stream?)
 						end
 						

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

@@ -25,7 +25,7 @@ module Async
 					false
 				end
 				
-				def write_interim_response(response)
+				def write_interim_response(status, headers = nil)
 				end
 				
 				def peer

data/lib/async/http/version.rb

@@ -5,6 +5,6 @@
 
 module Async
 	module HTTP
-		VERSION = "0.72.0"
+		VERSION = "0.73.0"
 	end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: async-http
 version: !ruby/object:Gem::Version
-  version: 0.72.0
+  version: 0.73.0
 platform: ruby
 authors:
 - Samuel Williams
@@ -58,7 +58,7 @@ cert_chain:
   Q2K9NVun/S785AP05vKkXZEFYxqG6EW012U4oLcFl5MySFajYXRYbuUpH6AY+HP8
   voD0MPg1DssDLKwXyt1eKD/+Fq0bFWhwVM/1XiAXL7lyYUyOq24KHgQ2Csg=
   -----END CERTIFICATE-----
-date: 2024-08-29 00:00:00.000000000 Z
+date: 2024-08-31 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: async
@@ -122,14 +122,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.29'
+        version: '0.30'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.29'
+        version: '0.30'
 - !ruby/object:Gem::Dependency
   name: protocol-http1
   requirement: !ruby/object:Gem::Requirement