async-http 0.70.0 → 0.72.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f049c61bc6afdaebb893a682209ac35e761efa9a4ef6aea5759e276153a4da80
-  data.tar.gz: 0334a7f914b12caafa97e7ab00ea3e02845c8451c52c159bbd625ecc9a8f9d51
+  metadata.gz: 8b358f54acd8cf0943e35f30865d7f505025bad37a2c37e0dda5f8438efe3b25
+  data.tar.gz: a8a9eee82cc0723cf8101ef727072d4518a38ddb7e68f5f0f2d15027e92e83a9
 SHA512:
-  metadata.gz: 93188a2d18c3991ac8e1b7306dc26ea552b7094b1683c3c6434ee2b76951c4fa725a280438026cd410ccd1229efd36bbf337173241688e61552a8c9c23f5d07c
-  data.tar.gz: e0cc2b44db653d9770facff7958a512bb56da5483b07348bc12af17b76fec7cfc22e95a2d3d5fdcb608edac47591304c04e6ab558e5ae07fff816d125d3e0379
+  metadata.gz: 7f5720d8ac42c48afb19bd5c387db2da015b2d210836c6d1a7abfe3cd3a4df85587c92b9bbdee61fce397302c66c8ca303c123d9e7f9a2201b26602ea0240e58
+  data.tar.gz: 91a543963515d1375953da096781b7b9d5a161e20bafc6aa067fbf0e09ff546ae3b48e18a5d8ca06dc5173184546898d415310b6bfbded5605040bce08443010

data/lib/async/http/client.rb

@@ -165,6 +165,10 @@ module Async
 						end
 						
 						super.tap do |response|
+							if version = response&.version
+								span['http.version'] = version
+							end
+							
 							if status = response&.status
 								span['http.status_code'] = status
 							end

data/lib/async/http/endpoint.rb

@@ -62,7 +62,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 alpn_protocols [Array<String>] the alpn protocols to negotiate.
+			# @option alpn_protocols [Array(String)] the alpn protocols to negotiate.
 			def initialize(url, endpoint = nil, **options)
 				super(**options)
 				

data/lib/async/http/middleware/location_redirector.rb

@@ -1,8 +1,7 @@
 # frozen_string_literal: true
 
 # Released under the MIT License.
-# Copyright, 2018-2023, by Samuel Williams.
-# Copyright, 2019-2020, by Brian Morearty.
+# Copyright, 2024, by Samuel Williams.
 
 require_relative '../reference'
 
@@ -82,17 +81,7 @@ module Async
 					# We don't want to follow redirects for HEAD requests:
 					return super if request.head?
 					
-					if body = request.body
-						if body.respond_to?(:rewind)
-							# The request body was already rewindable, so use it as is:
-							body = request.body
-						else
-							# The request body was not rewindable, and we might need to resubmit it if we get a response status of 307 or 308, so make it rewindable:
-							body = ::Protocol::HTTP::Body::Rewindable.new(body)
-							request.body = body
-						end
-					end
-					
+					body = ::Protocol::HTTP::Body::Rewindable.wrap(request)
 					hops = 0
 					
 					while hops <= @maximum_hops
@@ -141,4 +130,4 @@ module Async
 			end
 		end
 	end
-end
+end

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

@@ -7,6 +7,7 @@
 # Copyright, 2024, by Anton Zhuravsky.
 
 require_relative 'connection'
+require 'console/event/failure'
 
 module Async
 	module HTTP
@@ -17,8 +18,9 @@ module Async
 						@persistent = false
 						write_response(@version, status, {})
 						write_body(@version, nil)
-					rescue Errno::ECONNRESET, Errno::EPIPE
-						# Nothing we can do...
+					rescue => error
+						# At this point, there is very little we can do to recover:
+						Console::Event::Failure.for(error).emit(self, "Failed to write failure response.", severity: :debug)
 					end
 					
 					def next_request
@@ -33,13 +35,9 @@ module Async
 						end
 						
 						return request
-					rescue Async::TimeoutError
-						# For an interesting discussion about this behaviour, see https://trac.nginx.org/nginx/ticket/1005
-						# If you enable this, you will see some spec failures...
-						# fail_request(408)
-						raise
-					rescue
+					rescue ::Protocol::HTTP1::BadRequest
 						fail_request(400)
+						# Conceivably we could retry here, but we don't really know how bad the error is, so it's better to just fail:
 						raise
 					end
 					

data/lib/async/http/relative_location.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 # Released under the MIT License.
-# Copyright, 2018-2023, by Samuel Williams.
+# Copyright, 2018-2024, by Samuel Williams.
 # Copyright, 2019-2020, by Brian Morearty.
 
 require_relative 'middleware/location_redirector'

data/lib/async/http/server.rb

@@ -81,6 +81,7 @@ module Async
 					end
 					
 					attributes = {
+						'http.version': request.version,
 						'http.method': request.method,
 						'http.authority': request.authority,
 						'http.scheme': request.scheme,

data/lib/async/http/version.rb

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

data/readme.md

@@ -12,6 +12,14 @@ Please see the [project documentation](https://socketry.github.io/async-http/) f
 
   - [Testing](https://socketry.github.io/async-http/guides/testing/index) - This guide explains how to use `Async::HTTP` clients and servers in your tests.
 
+## See Also
+
+  - [benchmark-http](https://github.com/socketry/benchmark-http) — A benchmarking tool to report on web server concurrency.
+  - [falcon](https://github.com/socketry/falcon) — A rack compatible server built on top of `async-http`.
+  - [async-websocket](https://github.com/socketry/async-websocket) — Asynchronous client and server websockets.
+  - [async-rest](https://github.com/socketry/async-rest) — A RESTful resource layer built on top of `async-http`.
+  - [async-http-faraday](https://github.com/socketry/async-http-faraday) — A faraday adapter to use `async-http`.
+
 ## Contributing
 
 We welcome contributions to this project.
@@ -22,14 +30,6 @@ We welcome contributions to this project.
 4.  Push to the branch (`git push origin my-new-feature`).
 5.  Create new Pull Request.
 
-## See Also
-
-  - [benchmark-http](https://github.com/socketry/benchmark-http) — A benchmarking tool to report on web server concurrency.
-  - [falcon](https://github.com/socketry/falcon) — A rack compatible server built on top of `async-http`.
-  - [async-websocket](https://github.com/socketry/async-websocket) — Asynchronous client and server websockets.
-  - [async-rest](https://github.com/socketry/async-rest) — A RESTful resource layer built on top of `async-http`.
-  - [async-http-faraday](https://github.com/socketry/async-http-faraday) — A faraday adapter to use `async-http`.
-
 ### Developer Certificate of Origin
 
 In order to protect users of this project, we require all contributors to comply with the [Developer Certificate of Origin](https://developercertificate.org/). This ensures that all contributions are properly licensed and attributed.

metadata

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