async-htty 0.2.0 → 0.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c9d30babccdd7bf9fe410add7bb9dd2a6f7dafaa4d642bb377bf431f2aea4818
-  data.tar.gz: 8cd9da81dce5256dd99c9724491f07c94217fda9efd604f468c67b61a1b4f379
+  metadata.gz: f033a6514d7f18ec5f7c94b4f547ef2fa00e025eedce18d004387dde4baae5bb
+  data.tar.gz: 6d6a4cf76032a9e403c3f38f0ed4a46d1f6af61dd61ca3b712c68c93eabea6f8
 SHA512:
-  metadata.gz: b9a43f93f8c0a6a73e3114ea24ff950fcce09d49bd06752ca121e720ea8f566d220b41ee000514819bd1b537551e7ef74b0e191f3c6709e73a44083b0278e667
-  data.tar.gz: 0d9d94b0125c871ee18a5f95c98e45249e84b2397ef751f9c4b56b9b3175842b438630a998fd063fe0f848f75570c6a9f94b66ddc7044cdd99451173ff6c387d
+  metadata.gz: 1f027635ab2786cae7850b213eba3c37e3ac9762409d746a0d54e83d6a9366963d598843f7ccedbd729298c877181062dce7c28f20154c91a6a944cdea2430c3
+  data.tar.gz: b3bfefe4d878d7a404ffc3f3d66798f9acd5e6e17b03743849f58a281dde09a3d63c1fc85bb7918f4b8ff6a340c4335e2ddaaf1d20e882d1f270c6f953ba3f82

data/lib/async/htty/protocol/htty/server.rb

@@ -0,0 +1,24 @@
+# frozen_string_literal: true
+
+# Released under the MIT License.
+# Copyright, 2026, by Samuel Williams.
+
+require "async/http/protocol/http2/server"
+
+module Async
+	module HTTY
+		module Protocol
+			module HTTY
+				class Server < ::Async::HTTP::Protocol::HTTP2::Server
+					def receive_goaway(frame)
+						super
+
+						unless self.framer.nil?
+							self.send_goaway
+						end
+					end
+				end
+			end
+		end
+	end
+end

data/lib/async/htty/protocol/htty.rb

@@ -6,6 +6,7 @@
 require "protocol/htty"
 
 require "async/http/protocol/http2"
+require_relative "htty/server"
 
 module Async
 	module HTTY
@@ -24,7 +25,7 @@ module Async
 				def self.server(stream, settings: ::Async::HTTP::Protocol::HTTP2::SERVER_SETTINGS)
 					stream = ::Protocol::HTTY::Stream.open(stream, bootstrap: :write)
 					
-					server = ::Async::HTTP::Protocol::HTTP2::Server.new(stream)
+					server = Server.new(stream)
 					server.read_connection_preface(settings)
 					server.start_connection
 					

data/lib/async/htty/version.rb

@@ -5,6 +5,6 @@
 
 module Async
 	module HTTY
-		VERSION = "0.2.0"
+		VERSION = "0.2.1"
 	end
 end

data/readme.md

@@ -14,6 +14,11 @@ Please see the [project documentation](https://socketry.github.io/async-htty/) f
 
 Please see the [project releases](https://socketry.github.io/async-htty/releases/index) for all releases.
 
+### v0.2.1
+
+  - Send a server-side GOAWAY when the HTTY client closes an HTTP/2 session, allowing terminal clients to detach cleanly.
+  - Add PTY coverage for binary request/response bodies across the full byte range.
+
 ### v0.2.0
 
   - Reopen `stdin`, `stdout`, and `stderr` to null devices to prevent output from interfering with HTTY's byte stream.

data/releases.md

@@ -1,5 +1,10 @@
 # Releases
 
+## v0.2.1
+
+  - Send a server-side GOAWAY when the HTTY client closes an HTTP/2 session, allowing terminal clients to detach cleanly.
+  - Add PTY coverage for binary request/response bodies across the full byte range.
+
 ## v0.2.0
 
   - Reopen `stdin`, `stdout`, and `stderr` to null devices to prevent output from interfering with HTTY's byte stream.

data/test/async/htty/protocol/htty.rb

@@ -4,12 +4,48 @@
 # Copyright, 2026, by Samuel Williams.
 
 require "async"
+require "protocol/http/body/writable"
 require "protocol/http/request"
 require "protocol/http/response"
 require "protocol/http2"
+require "protocol/http2/client"
+require "protocol/http2/stream"
+require "rbconfig"
 require "async/htty"
 
+require "async/htty/pty_stream"
+
+class EchoResponseStream < Protocol::HTTP2::Stream
+	attr :response_headers
+	attr :body
+	
+	def initialize(...)
+		super
+		@response_headers = []
+		@body = +"".b
+	end
+	
+	def process_headers(frame)
+		@response_headers = super
+	end
+	
+	def process_data(frame)
+		data = super
+		@body << data.b if data
+		return data
+	end
+end
+
+class EchoClient < Protocol::HTTP2::Client
+	def create_stream(id = next_stream_id)
+		EchoResponseStream.create(self, id)
+	end
+end
+
 describe Async::HTTY::Protocol::HTTY do
+	let(:root) {File.expand_path("../../../..", __dir__)}
+	let(:ruby_load_path) {File.join(root, "lib")}
+	
 	def make_pipes
 		server_input, client_output = IO.pipe
 		client_input, server_output = IO.pipe
@@ -36,6 +72,29 @@ describe Async::HTTY::Protocol::HTTY do
 		IO::Stream::Duplex(pipes[:server_input], pipes[:server_output])
 	end
 	
+	def spawn_fixture(name)
+		environment = {
+			"HTTY" => "1",
+			"RUBYLIB" => [ruby_load_path, ENV["RUBYLIB"]].compact.join(":"),
+		}
+		executable = File.join(root, "fixtures", "async", "htty", "executables", name)
+		
+		PTY.spawn(environment, RbConfig.ruby, executable)
+	end
+	
+	def with_fixture(name)
+		input, output, pid = spawn_fixture(name)
+		input.binmode
+		output.binmode
+		
+		stream = Async::HTTY::PTYStream.new(input, output)
+		
+		yield stream
+	ensure
+		stream&.close
+		Process.wait(pid) rescue nil
+	end
+	
 	it "can carry an HTTP/2 request over HTTY bootstrap and raw transport" do
 		pipes = make_pipes
 		
@@ -60,6 +119,95 @@ describe Async::HTTY::Protocol::HTTY do
 		end
 	end
 	
+	it "round trips all byte values over a real PTY" do
+		payload = (0x00..0xff).to_a.pack("C*")
+		
+		with_fixture("echo_body.rb") do |stream|
+			Protocol::HTTY::Stream.new(stream).read_bootstrap
+			
+			framer = Protocol::HTTP2::Framer.new(stream)
+			client = EchoClient.new(framer)
+			client.send_connection_preface
+			
+			request = client.create_stream
+			request.send_headers(
+				[
+					[":method", "POST"],
+					[":path", "/echo"],
+					[":scheme", "http"],
+					[":authority", "htty.local"],
+					["content-length", payload.bytesize.to_s],
+					["content-type", "application/octet-stream"],
+				]
+			)
+			request.send_data(payload)
+			request.send_data(nil)
+			
+			until request.closed?
+				client.read_frame
+			end
+			
+			expect(request.response_headers.to_h[":status"]).to be == "200"
+			expect(request.body).to be == payload
+		ensure
+			client&.send_goaway
+			client&.close
+		end
+	end
+	
+	it "returns from accept when the client sends GOAWAY while a response body is active" do
+		pipes = make_pipes
+		body = Protocol::HTTP::Body::Writable.new
+		
+		Sync do |task|
+			request_started = Async::Notification.new
+			server_finished = Async::Notification.new
+			
+			server = Async::HTTY::Server.for do |request|
+				request_started.signal
+				Protocol::HTTP::Response[200, {}, body]
+			end
+			
+			server_task = task.async do
+				server.accept(server_stream(pipes))
+			ensure
+				server_finished.signal
+			end
+			
+			stream = Protocol::HTTY::Stream.open(client_stream(pipes), bootstrap: :read)
+			framer = Protocol::HTTP2::Framer.new(stream)
+			client = Protocol::HTTP2::Client.new(framer)
+			client.send_connection_preface
+			
+			request = client.create_stream
+			request.send_headers(
+				[[":method", "GET"], [":path", "/"], [":scheme", "http"], [":authority", "htty.local"]],
+				Protocol::HTTP2::END_STREAM
+			)
+			
+			request_started.wait
+			client.send_goaway
+			
+			task.with_timeout(1) do
+				server_finished.wait
+			end
+			
+			frame = task.with_timeout(1) do
+				loop do
+					frame = framer.read_frame(client.local_settings.maximum_frame_size)
+					break frame if frame.is_a?(Protocol::HTTP2::GoawayFrame)
+				end
+			end
+			
+			expect(frame).to be(:is_a?, Protocol::HTTP2::GoawayFrame)
+		ensure
+			body.close
+			client&.close
+			server_task&.stop
+			close_pipes(pipes)
+		end
+	end
+	
 	it "ignores terminal noise before the HTTY bootstrap" do
 		pipes = make_pipes
 		

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: async-htty
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.2.1
 platform: ruby
 authors:
 - Samuel Williams
@@ -105,6 +105,7 @@ files:
 - lib/async/htty/error.rb
 - lib/async/htty/protocol.rb
 - lib/async/htty/protocol/htty.rb
+- lib/async/htty/protocol/htty/server.rb
 - lib/async/htty/server.rb
 - lib/async/htty/version.rb
 - license.md