Support io-endpoint gem.

Author: ioquatix

async-http.gemspec

@@ -20,8 +20,8 @@ Gem::Specification.new do |spec|
 	spec.required_ruby_version = ">= 3.0"
 
 	spec.add_dependency "async", ">= 1.25"
-	spec.add_dependency "async-io", ">= 1.28"
 	spec.add_dependency "async-pool", ">= 0.2"
+	spec.add_dependency "io-endpoint", "~> 0.6"
 	spec.add_dependency "protocol-http", "~> 0.26.0"
 	spec.add_dependency "protocol-http1", "~> 0.18.0"
 	spec.add_dependency "protocol-http2", "~> 0.16.0"

examples/google/codeotaku.rb

@@ -12,8 +12,6 @@
 URL = "https://www.codeotaku.com/index"
 ENDPOINT = Async::HTTP::Endpoint.parse(URL)
 
-Console.logger.enable(Async::IO::Stream, Console::Logger::DEBUG)
-
 if count = ENV['COUNT']&.to_i
 	terms = terms.first(count)
 end

examples/google/search.rb

@@ -12,8 +12,6 @@
 URL = "https://www.google.com/search"
 ENDPOINT = Async::HTTP::Endpoint.parse(URL)
 
-# Console.logger.enable(Async::IO::Stream, Console::Logger::DEBUG)
-
 class Google < Protocol::HTTP::Middleware
 	def search(term)
 		Console.logger.info(self) {"Searching for #{term}..."}

fixtures/async/http/a_protocol.rb

@@ -481,8 +481,10 @@ def after
 
 				it "can't get /" do
 					expect do
-						client.get("/")
-					end.to raise_exception(Async::TimeoutError)
+						Console.debug(self) {"Connecting to #{endpoint.inspect}"}
+						response = client.get("/")
+						Console.debug(self) {"Got response #{response.inspect}"}
+					end.to raise_exception(::IO::TimeoutError)
 				end
 			end
 
@@ -542,33 +544,41 @@ def around
 					Console.logger.level = current
 				end
 
+				def timeout = nil
+				
 				it "doesn't cancel all requests" do
-					tasks = []
 					task = Async::Task.current
+					tasks = []
 					stopped = []
 
 					10.times do
-						tasks << task.async {
-							begin
-								loop do
-									client.get('http://127.0.0.1:8080/a').finish
-								end
+						task.async do |child|
+							tasks << child
+							
+							loop do
+								response = client.get('/a')
+								response.finish
 							ensure
-								stopped << 'a'
+								response&.close
 							end
-						}
+						ensure
+							stopped << 'a'
+						end
 					end
 
 					10.times do
-						tasks << task.async {
-							begin
-								loop do
-									client.get('http://127.0.0.1:8080/b').finish
-								end
+						task.async do |child|
+							tasks << child
+							
+							loop do
+								response = client.get('/b')
+								response.finish
 							ensure
-								stopped << 'b'
+								response&.close
 							end
-						}
+						ensure
+							stopped << 'b'
+						end
 					end
 
 					tasks.each do |child|

gems.rb

@@ -8,8 +8,9 @@
 gemspec
 
 # gem "async", path: "../async"
-# gem "async-io", path: "../async-io"
 # gem "traces", path: "../traces"
+gem "io-endpoint", path: "../io-endpoint"
+# gem "sus-fixtures-async-http", path: "../sus-fixtures-async-http"
 
 # gem "protocol-http", path: "../protocol-http"
 # gem "protocol-http1", path: "../protocol-http1"
@@ -28,7 +29,7 @@
 	gem "covered"
 	gem "sus"
 	gem "sus-fixtures-async"
-	gem "sus-fixtures-async-http", "~> 0.7"
+	gem "sus-fixtures-async-http", "~> 0.8"
 	gem "sus-fixtures-openssl"
 
 	gem "bake"

lib/async/http/body/pipe.rb

@@ -18,10 +18,7 @@ def initialize(input, output = Writable.new, task: Task.current)
 					@input = input
 					@output = output
 
-					head, tail = IO::Socket.pair(Socket::AF_UNIX, Socket::SOCK_STREAM)
-					
-					@head = IO::Stream.new(head)
-					@tail = tail
+					@head, @tail = ::Socket.pair(Socket::AF_UNIX, Socket::SOCK_STREAM)
 
 					@reader = nil
 					@writer = nil
@@ -68,7 +65,7 @@ def writer(task)
 
 					task.annotate "#{self.class} writer."
 
-					while chunk = @head.read_partial
+					while chunk = @head.readpartial(1024)
 						@output.write(chunk)
 					end
 				ensure

lib/async/http/client.rb

@@ -93,7 +93,6 @@ def call(request)
 
 					# 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
-					
 					return response
 				rescue Protocol::RequestFailed
 					# This is a specific case where the entire request wasn't sent before a failure occurred. So, we can even resend non-idempotent requests.
@@ -119,7 +118,9 @@ def call(request)
 						raise
 					end
 				ensure
-					@pool.release(connection) if connection
+					if connection
+						@pool.release(connection)
+					end
 				end
 			end
 

lib/async/http/endpoint.rb

@@ -4,18 +4,18 @@
 # Copyright, 2019-2023, by Samuel Williams.
 # Copyright, 2021-2022, by Adam Daniels.
 
-require 'async/io/host_endpoint'
-require 'async/io/ssl_endpoint'
-require 'async/io/ssl_socket'
-require 'async/io/shared_endpoint'
+require 'io/endpoint'
+require 'io/endpoint/host_endpoint'
+require 'io/endpoint/ssl_endpoint'
+require 'io/endpoint/shared_endpoint'
 
 require_relative 'protocol/http1'
 require_relative 'protocol/https'
 
 module Async
 	module HTTP
 		# Represents a way to connect to a remote HTTP server.
-		class Endpoint < Async::IO::Endpoint
+		class Endpoint < ::IO::Endpoint::Generic
 			def self.parse(string, endpoint = nil, **options)
 				url = URI.parse(string).normalize
 
@@ -164,7 +164,7 @@ def build_endpoint(endpoint = nil)
 
 				if secure?
 					# Wrap it in SSL:
-					return Async::IO::SSLEndpoint.new(endpoint,
+					return ::IO::Endpoint::SSLEndpoint.new(endpoint,
 						ssl_context: self.ssl_context,
 						hostname: @url.hostname,
 						timeout: self.timeout,
@@ -226,7 +226,7 @@ def tcp_options
 			end
 
 			def tcp_endpoint
-				Async::IO::Endpoint.tcp(self.hostname, port, **tcp_options)
+				::IO::Endpoint.tcp(self.hostname, port, **tcp_options)
 			end
 		end
 	end

lib/async/http/protocol/http1.rb

@@ -21,15 +21,11 @@ def self.trailer?
 				end
 
 				def self.client(peer)
-					stream = IO::Stream.new(peer, sync: true)
-					
-					return HTTP1::Client.new(stream, VERSION)
+					return HTTP1::Client.new(peer, VERSION)
 				end
 
 				def self.server(peer)
-					stream = IO::Stream.new(peer, sync: true)
-					
-					return HTTP1::Server.new(stream, VERSION)
+					return HTTP1::Server.new(peer, VERSION)
 				end
 
 				def self.names

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

@@ -59,6 +59,7 @@ def call(request, task: Task.current)
 							write_body(@version, body, false, trailer)
 						end
 
+						Console.logger.debug(self) {"Waiting for response..."}
 						response = Response.read(self, request)
 						@ready = true