Stop setting SERVER_PORT env key if nil

Author: Francisco Mejia

lib/protocol/rack/adapter/generic.rb

@@ -106,6 +106,9 @@ def unwrap_request(request, env)
 					if peer = request.peer
 						env[CGI::REMOTE_ADDR] = peer.ip_address
 					end
+
+					# SERVER_PORT should not be set if it is nil
+					env.delete(CGI::SERVER_PORT) if env[CGI::SERVER_PORT].nil?
 				end
 
 				def make_environment(request)

test/protocol/rack/adapter/generic.rb

@@ -35,6 +35,18 @@
 			expect(env).to have_keys("CONTENT_TYPE" => be == "text/plain")
 		end
 	end
+
+	with "app server without SERVER_PORT" do
+    let(:request) do
+      Protocol::HTTP::Request.new("https", "example.com", "GET", "/", "http/1.1", Protocol::HTTP::Headers[{"accept" => "text/html"}], nil)
+    end
+
+    it "does not include SERVER_PORT in the Rack environment" do
+      env = adapter.make_environment(request)
+      
+      expect(env).not.to have_keys(Protocol::Rack::CGI::SERVER_PORT)
+    end
+  end
 
 	with "a app that returns nil" do
 		include DisableConsoleContext