protocol-rack 0.15.0 → 0.16.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a0994443c638a7cf147afcb7a8b88a1a6e5e8ab607071c10792d65c912fb9b37
-  data.tar.gz: 22a6c27089e34db868a7af15fac1061979791872c811aef51762b68435fb9346
+  metadata.gz: 6d1c1680392355d0bf9a1aa9b6b76f2267f9d1f612e4ae194fe5933a95fb4486
+  data.tar.gz: 8401fd53bbe49d9547f187f06a23ba334eae3814570f041b333c8c3c30aa317b
 SHA512:
-  metadata.gz: 60caa82495eb534bfb3367009b7567304c1c36b2db646fbee99d7c68eaba810c9913ad8c847c7939e0d60bb29bd049bdfb67ff365f68f9142b65d6cf7a7bf8f8
-  data.tar.gz: 63313385d9603b0bdaddeb39410dbcf7a3485652478d886ffbb5e8ea926a1b5006bcbd21bdc3b6a3851e43f1f601ded523021d700cd00dd0d2ab252b529378fa
+  metadata.gz: 7e3194c1c3ae772b58d15126fa918150b3e04791bfcf24a25ef6314d573392c3b8fdfca9efd7bab47a17ab1fd99d2b9a32323699014c900840024368fdfcabcd
+  data.tar.gz: ebbcaf7b1368633de01b261c1ad8c6f741f798b74d28070e33d4023887b306acb9a86df5b243f2b3bef737a148663c45a94bd981e6cf3d9757ce82048c4be830

data/lib/protocol/rack/adapter/generic.rb

@@ -114,7 +114,7 @@ module Protocol
 					
 					if request.respond_to?(:hijack?) and request.hijack?
 						env[RACK_IS_HIJACK] = true
-						env[RACK_HIJACK] = proc{request.hijack!.io.dup}
+						env[RACK_HIJACK] = proc{request.hijack!.io}
 					end
 					
 					# HTTP/2 prefers `:authority` over `host`, so we do this for backwards compatibility.
@@ -188,7 +188,7 @@ module Protocol
 						# This is the newer mechanism for protocol upgrade:
 						if env["rack.protocol"]
 							headers["rack.protocol"] = protocol
-						
+							
 						# Older mechanism for protocol upgrade:
 						elsif env[CGI::HTTP_UPGRADE]
 							headers["upgrade"] = protocol

data/lib/protocol/rack/adapter/rack2.rb

@@ -44,7 +44,7 @@ module Protocol
 						RACK_INPUT => Input.new(request.body),
 						RACK_ERRORS => $stderr,
 						RACK_LOGGER => self.logger,
-
+						
 						# The HTTP request method, such as "GET" or "POST". This cannot ever be an empty string, and so is always required.
 						CGI::REQUEST_METHOD => request.method,
 						
@@ -55,7 +55,7 @@ module Protocol
 						CGI::PATH_INFO => request_path,
 						CGI::REQUEST_PATH => request_path,
 						CGI::REQUEST_URI => request.path,
-
+						
 						# The portion of the request URL that follows the ?, if any. May be empty, but is always required!
 						CGI::QUERY_STRING => query_string || "",
 						
@@ -68,7 +68,7 @@ module Protocol
 						# I'm not sure what sane defaults should be here:
 						CGI::SERVER_NAME => server_name,
 					}
-
+					
 					# SERVER_PORT is optional but must not be set if it is not present.
 					if server_port
 						env[CGI::SERVER_PORT] = server_port

data/lib/protocol/rack/body/enumerable.rb

@@ -100,7 +100,7 @@ module Protocol
 				def stream?
 					!@body.respond_to?(:each)
 				end
-
+				
 				# Stream the response body to the given stream.
 				# The body is automatically closed after streaming.
 				# 
@@ -112,7 +112,7 @@ module Protocol
 				ensure
 					self.close(error)
 				end
-
+				
 				# Read the next chunk from the response body.
 				# Returns nil when there are no more chunks.
 				# 

data/lib/protocol/rack/request.rb

@@ -23,13 +23,13 @@ module Protocol
 			def self.[](env)
 				env["protocol.http.request"] ||= new(env)
 			end
-
+			
 			# Initialize a new Request instance from a Rack environment.
 			# 
 			# @parameter env [Hash] The Rack environment hash.
 			def initialize(env)
 				@env = env
-
+				
 				super(
 					@env["rack.url_scheme"],
 					@env["HTTP_HOST"],
@@ -41,7 +41,7 @@ module Protocol
 					self.class.protocol(@env)
 				)
 			end
-
+			
 			# Extract the protocol list from the Rack environment.
 			# Checks both `rack.protocol` and `HTTP_UPGRADE` headers.
 			# 
@@ -54,7 +54,7 @@ module Protocol
 					return protocols.split(/\s*,\s*/)
 				end
 			end
-
+			
 			# Extract HTTP headers from the Rack environment.
 			# Converts Rack's `HTTP_*` environment variables to proper HTTP headers.
 			# 
@@ -68,7 +68,7 @@ module Protocol
 						headers[key[5..-1].gsub("_", "-").downcase] = value
 					end
 				end
-
+				
 				return headers
 			end
 		end

data/lib/protocol/rack/response.rb

@@ -45,11 +45,11 @@ module Protocol
 				unless ignored.empty?
 					Console.warn(self, "Ignoring hop headers!", ignored: ignored)
 				end
-
+				
 				if hijack_body = meta["rack.hijack"]
 					body = hijack_body
 				end
-
+				
 				body = Body.wrap(env, status, headers, body, request&.body, request&.head?)
 				
 				protocol = meta[RACK_PROTOCOL]

data/lib/protocol/rack/version.rb

@@ -5,6 +5,6 @@
 
 module Protocol
 	module Rack
-		VERSION = "0.15.0"
+		VERSION = "0.16.0"
 	end
 end

data/readme.md

@@ -67,6 +67,10 @@ run proc{|env|
 
 Please see the [project releases](https://socketry.github.io/protocol-rack/releases/index) for all releases.
 
+### v0.16.0
+
+  - Hijacked IO is no longer duped, as it's not retained by the original connection, and `SSLSocket` does not support duping.
+
 ### v0.15.0
 
   - Use `IO::Stream::Readable` for the input body, which is a better tested and more robust interface.

data/releases.md

@@ -1,5 +1,9 @@
 # Releases
 
+## v0.16.0
+
+  - Hijacked IO is no longer duped, as it's not retained by the original connection, and `SSLSocket` does not support duping.
+
 ## v0.15.0
 
   - Use `IO::Stream::Readable` for the input body, which is a better tested and more robust interface.

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: protocol-rack
 version: !ruby/object:Gem::Version
-  version: 0.15.0
+  version: 0.16.0
 platform: ruby
 authors:
 - Samuel Williams