Don't assign nil to rack.protocol. (#19)

Author: ioquatix

lib/protocol/rack/adapter/generic.rb

@@ -74,6 +74,11 @@ def unwrap_headers(headers, env)
 				# @parameter request [Protocol::HTTP::Request] The incoming request.
 				# @parameter env [Hash] The rack `env`.
 				def unwrap_request(request, env)
+					# The request protocol, either from the upgrade header or the HTTP/2 pseudo header of the same name.
+					if protocol = request.protocol
+						env[RACK_PROTOCOL] = protocol
+					end
+					
 					if content_type = request.headers.delete("content-type")
 						env[CGI::CONTENT_TYPE] = content_type
 					end

lib/protocol/rack/adapter/rack2.rb

@@ -37,9 +37,6 @@ def make_environment(request)
 						RACK_ERRORS => $stderr,
 						RACK_LOGGER => self.logger,
 
-						# The request protocol, either from the upgrade header or the HTTP/2 pseudo header of the same name.
-						RACK_PROTOCOL => request.protocol,
-						
 						# 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,
 

lib/protocol/rack/adapter/rack3.rb

@@ -30,9 +30,6 @@ def make_environment(request)
 						RACK_ERRORS => $stderr,
 						RACK_LOGGER => self.logger,
 
-						# The request protocol, either from the upgrade header or the HTTP/2 pseudo header of the same name.
-						RACK_PROTOCOL => request.protocol,
-						
 						# The response finished callbacks:
 						RACK_RESPONSE_FINISHED => [],
 

lib/protocol/rack/adapter/rack31.rb

@@ -21,9 +21,6 @@ def make_environment(request)
 						RACK_ERRORS => $stderr,
 						RACK_LOGGER => self.logger,
 
-						# The request protocol, either from the upgrade header or the HTTP/2 pseudo header of the same name.
-						RACK_PROTOCOL => request.protocol,
-						
 						# The response finished callbacks:
 						RACK_RESPONSE_FINISHED => [],