protocol-http1 0.40.2 → 0.41.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fff1a36300fb1ea0775689411e9b86a45093e6d064d10868c419e38422754bea
-  data.tar.gz: c5204b0d2f701c834685b84c405ae1a98da109ec8ccdaf0c6d4a99a04b489ccb
+  metadata.gz: aa2e943387ba60e2d581cbdebf2dbd4ae864fb848965f30dff7e383a684a731f
+  data.tar.gz: 1824ea1d12202dd3d5dc65192229492a4ad45900dabdbe2b311677878d91866c
 SHA512:
-  metadata.gz: 454ec4c8703447ad07cc199d482210d40b3abd79961b456eb1d0680869e35f87b72a0f0ba7bea5739178a40130007397b3d71fe9f467c83507628c2625e0cc98
-  data.tar.gz: f0c33d4fb8815e1f46e447ab5c693da0baaed0cf08e288d05030157b4bdaf6e792f306efbdaa3db21b80e1d1731d624647c4b3c13e5e5b308deac7b7bbf58346
+  metadata.gz: 5fac180bc79a47ad08969214bf46c6fef9c431608e643f6a707cf479fc3f4617f08e87b0057be350d768ad1151107f9147a6a35cd403a70e227d6dde2cbb14be
+  data.tar.gz: 76e0fd036db9afcddccc25f49a8dac62019f3f43e8710d8aa66ca73d2179a2da0d8efb6d3d7ddbd12032b6368fa0de2949e21613149e4f38fd7b8f53cb2a6734

data/lib/protocol/http1/connection.rb

@@ -177,15 +177,15 @@ module Protocol
 			# Write the appropriate header for connection persistence.
 			def write_connection_header(version)
 				if version == HTTP10
-					@stream.write("connection: keep-alive\r\n") if @persistent
+					write_stream("connection: keep-alive\r\n") if @persistent
 				else
-					@stream.write("connection: close\r\n") unless @persistent
+					write_stream("connection: close\r\n") unless @persistent
 				end
 			end
 			
 			# Write the appropriate header for connection upgrade.
 			def write_upgrade_header(upgrade)
-				@stream.write("connection: upgrade\r\nupgrade: #{upgrade}\r\n")
+				write_stream("connection: upgrade\r\nupgrade: #{upgrade}\r\n")
 			end
 			
 			# Indicates whether the connection has been hijacked meaning its IO has been handed over and is not usable anymore.
@@ -262,8 +262,8 @@ module Protocol
 			def write_request(authority, method, target, version, headers)
 				open!
 				
-				@stream.write("#{method} #{target} #{version}\r\n")
-				@stream.write("host: #{authority}\r\n") if authority
+				write_stream("#{method} #{target} #{version}\r\n")
+				write_stream("host: #{authority}\r\n") if authority
 				
 				write_headers(headers)
 			rescue
@@ -284,7 +284,7 @@ module Protocol
 				end
 				
 				# Safari WebSockets break if no reason is given:
-				@stream.write("#{version} #{status} #{reason}\r\n")
+				write_stream("#{version} #{status} #{reason}\r\n")
 				
 				write_headers(headers)
 			end
@@ -303,12 +303,12 @@ module Protocol
 					raise ProtocolError, "Cannot write interim response in state: #{@state}!"
 				end
 				
-				@stream.write("#{version} #{status} #{reason}\r\n")
+				write_stream("#{version} #{status} #{reason}\r\n")
 				
 				write_headers(headers)
 				
-				@stream.write("\r\n")
-				@stream.flush
+				write_stream("\r\n")
+				flush_stream
 			end
 			
 			# Write headers to the connection.
@@ -331,7 +331,7 @@ module Protocol
 					end
 					
 					# Write it:
-					@stream.write("#{name}: #{value}\r\n")
+					write_stream("#{name}: #{value}\r\n")
 				end
 			end
 			
@@ -340,6 +340,8 @@ module Protocol
 			# @parameter length [Integer] the maximum number of bytes to read.
 			def readpartial(length)
 				@stream.readpartial(length)
+			rescue Errno::EPIPE, Errno::ECONNRESET
+				raise HTTP::RemoteError, "Remote connection closed during readpartial!"
 			end
 			
 			# Read some data from the connection.
@@ -347,6 +349,8 @@ module Protocol
 			# @parameter length [Integer] the number of bytes to read.
 			def read(length)
 				@stream.read(length)
+			rescue Errno::EPIPE, Errno::ECONNRESET
+				raise HTTP::RemoteError, "Remote connection closed during read!"
 			end
 			
 			# Read a line from the connection.
@@ -541,16 +545,16 @@ module Protocol
 				
 				write_upgrade_header(protocol)
 				
-				@stream.write("\r\n")
-				@stream.flush # Don't remove me!
+				write_stream("\r\n")
+				flush_stream # Don't remove me!
 				
 				if body
 					body.each do |chunk|
-						@stream.write(chunk)
-						@stream.flush
+						write_stream(chunk)
+						flush_stream
 					end
 					
-					@stream.close_write
+					close_write_stream
 				end
 				
 				return @stream
@@ -572,16 +576,16 @@ module Protocol
 				
 				write_connection_header(version)
 				
-				@stream.write("\r\n")
-				@stream.flush # Don't remove me!
+				write_stream("\r\n")
+				flush_stream # Don't remove me!
 				
 				if body
 					body.each do |chunk|
-						@stream.write(chunk)
-						@stream.flush
+						write_stream(chunk)
+						flush_stream
 					end
 					
-					@stream.close_write
+					close_write_stream
 				end
 				
 				return @stream
@@ -595,8 +599,8 @@ module Protocol
 			#
 			# @parameter body [Object | Nil] the body to write.
 			def write_empty_body(body = nil)
-				@stream.write("content-length: 0\r\n\r\n")
-				@stream.flush
+				write_stream("content-length: 0\r\n\r\n")
+				flush_stream
 				
 				body&.close
 			ensure
@@ -612,12 +616,12 @@ module Protocol
 			# @parameter head [Boolean] whether the request was a `HEAD` request.
 			# @raises [ContentLengthError] if the body length does not match the content length specified.
 			def write_fixed_length_body(body, length, head)
-				@stream.write("content-length: #{length}\r\n\r\n")
+				write_stream("content-length: #{length}\r\n\r\n")
 				
 				if head
-					@stream.flush
+					flush_stream
 				else
-					@stream.flush unless body.ready?
+					flush_stream unless body.ready?
 					
 					chunk_length = 0
 					# Use a manual read loop (not body.each) so that body.close runs after the response is fully written and flushed. This ensures completion callbacks (e.g. rack.response_finished) don't delay the client.
@@ -628,11 +632,11 @@ module Protocol
 							raise ContentLengthError, "Trying to write #{chunk_length} bytes, but content length was #{length} bytes!"
 						end
 						
-						@stream.write(chunk)
-						@stream.flush unless body.ready?
+						write_stream(chunk)
+						flush_stream unless body.ready?
 					end
 					
-					@stream.flush
+					flush_stream
 					
 					if chunk_length != length
 						raise ContentLengthError, "Wrote #{chunk_length} bytes, but content length was #{length} bytes!"
@@ -657,33 +661,33 @@ module Protocol
 			# @parameter head [Boolean] whether the request was a `HEAD` request.
 			# @parameter trailer [Hash | Nil] the trailers to write.
 			def write_chunked_body(body, head, trailer = nil)
-				@stream.write("transfer-encoding: chunked\r\n\r\n")
+				write_stream("transfer-encoding: chunked\r\n\r\n")
 				
 				if head
-					@stream.flush
+					flush_stream
 				else
-					@stream.flush unless body.ready?
+					flush_stream unless body.ready?
 					
 					# Use a manual read loop (not body.each) so that body.close runs after the terminal chunk is written. With body.each, the ensure { close } fires before the terminal "0\r\n\r\n" is sent, delaying the client.
 					while chunk = body.read
 						next if chunk.size == 0
 						
-						@stream.write("#{chunk.bytesize.to_s(16).upcase}\r\n")
-						@stream.write(chunk)
-						@stream.write(CRLF)
+						write_stream("#{chunk.bytesize.to_s(16).upcase}\r\n")
+						write_stream(chunk)
+						write_stream(CRLF)
 						
-						@stream.flush unless body.ready?
+						flush_stream unless body.ready?
 					end
 					
 					if trailer&.any?
-						@stream.write("0\r\n")
+						write_stream("0\r\n")
 						write_headers(trailer)
-						@stream.write("\r\n")
+						write_stream("\r\n")
 					else
-						@stream.write("0\r\n\r\n")
+						write_stream("0\r\n\r\n")
 					end
 					
-					@stream.flush
+					flush_stream
 				end
 			rescue => error
 				raise
@@ -702,20 +706,20 @@ module Protocol
 				# We can't be persistent because we don't know the data length:
 				@persistent = false
 				
-				@stream.write("\r\n")
+				write_stream("\r\n")
 				
 				unless head
-					@stream.flush unless body.ready?
+					flush_stream unless body.ready?
 					
 					while chunk = body.read
-						@stream.write(chunk)
+						write_stream(chunk)
 						
-						@stream.flush unless body.ready?
+						flush_stream unless body.ready?
 					end
 				end
 				
-				@stream.flush
-				@stream.close_write
+				flush_stream
+				close_write_stream
 			rescue => error
 				raise
 			ensure
@@ -1026,6 +1030,26 @@ module Protocol
 					return read_remainder_body
 				end
 			end
+			
+			private
+			
+			def write_stream(data)
+				@stream.write(data)
+			rescue Errno::EPIPE, Errno::ECONNRESET
+				raise HTTP::RemoteError, "Remote connection closed during write!"
+			end
+			
+			def flush_stream
+				@stream.flush
+			rescue Errno::EPIPE, Errno::ECONNRESET
+				raise HTTP::RemoteError, "Remote connection closed during flush!"
+			end
+			
+			def close_write_stream
+				@stream.close_write
+			rescue Errno::EPIPE, Errno::ECONNRESET
+				raise HTTP::RemoteError, "Remote connection closed during close_write!"
+			end
 		end
 	end
 end

data/lib/protocol/http1/version.rb

@@ -5,6 +5,6 @@
 
 module Protocol
 	module HTTP1
-		VERSION = "0.40.2"
+		VERSION = "0.41.0"
 	end
 end

data/readme.md

@@ -30,6 +30,10 @@ Please see the [project documentation](https://socketry.github.io/protocol-http1
 
 Please see the [project releases](https://socketry.github.io/protocol-http1/releases/index) for all releases.
 
+### v0.41.0
+
+  - Map remote socket failures to `Protocol::HTTP::RemoteError` and prevent connection reuse.
+
 ### v0.40.2
 
   - Handle unexpected EOF while reading chunked trailers.
@@ -67,10 +71,6 @@ Please see the [project releases](https://socketry.github.io/protocol-http1/rele
   - Tidy up implementation of `read_line?` to handle line length errors and protocol violations more clearly.
   - Improve error handling for unexpected connection closures (`Errno::ECONNRESET`) in `read_line?`.
 
-### v0.35.0
-
-  - Add traces provider for `Protocol::HTTP1::Connection`.
-
 ## Contributing
 
 We welcome contributions to this project.

data/releases.md

@@ -1,5 +1,9 @@
 # Releases
 
+## v0.41.0
+
+  - Map remote socket failures to `Protocol::HTTP::RemoteError` and prevent connection reuse.
+
 ## v0.40.2
 
   - Handle unexpected EOF while reading chunked trailers.

data.tar.gz.sig

@@ -1,2 +1,4 @@
-
��~��CP�9
-���'��uxϦ���Ѳ���(K�AQ��k�/FwR�����~��<���%���P����@��hz\�[� 
+@�&qq�$q���:L�vI��~�_x��q������2"�t�
+�
+��ԩ��c�Sx��(W���G�C���mH��0��%:W��<�$iAֶ��5��c��,�c�'�)>Ʒ������-�Ug�d+�T��ޚVC��'��08�qƇ�\%:�3����1�<R�,�P��d@	��kF����b�I��d�(�sSTckG�s�T��>i�!hN��I��n;�5�ŧ1I�nhz�e����W)؟��Hr�)$%v�ܱ�mK�b��u�gƔu]���F���"s0n�u�a�Υ؏�)����ɐh���ԭ���c���(y�k��P?���uZ��h�~�zr�s��*�,�J=�
+Q��rJ~�4[d7N/,;�e�.sɱ[

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: protocol-http1
 version: !ruby/object:Gem::Version
-  version: 0.40.2
+  version: 0.41.0
 platform: ruby
 authors:
 - Samuel Williams