Improve compatibility of non-string header values. (#7)

* Add failing test case.

* Ignore non-string values.

Author: ioquatix

.github/workflows/coverage.yaml

@@ -14,7 +14,7 @@ jobs:
     name: ${{matrix.ruby}} on ${{matrix.os}}
     runs-on: ${{matrix.os}}-latest
     env:
-      BUNDLER_GEMFILE: ${{matrix.gemfile}}
+      BUNDLE_GEMFILE: ${{matrix.gemfile}}
 
     strategy:
       matrix:

.github/workflows/test.yaml

@@ -14,7 +14,7 @@ jobs:
     runs-on: ${{matrix.os}}-latest
     continue-on-error: ${{matrix.experimental}}
     env:
-      BUNDLER_GEMFILE: ${{matrix.gemfile}}
+      BUNDLE_GEMFILE: ${{matrix.gemfile}}
 
     strategy:
       matrix:

.gitignore

@@ -3,3 +3,5 @@
 /gems.locked
 /.covered.db
 /external
+
+*.lock

gems/gems.rb

@@ -0,0 +1,8 @@
+source "https://rubygems.org"
+
+gem "async-http"
+gem "bake-test"
+gem "bake-test-external"
+gem "sus"
+gem "sus-fixtures-async-http"
+gem "covered"

gems/rack-v1.rb

@@ -3,6 +3,6 @@
 # Released under the MIT License.
 # Copyright, 2022, by Samuel Williams.
 
-eval_gemfile("../gems.rb")
+eval_gemfile("gems.rb")
 
 gem "rack", "~> 1.0"

gems/rack-v2.rb

@@ -3,6 +3,6 @@
 # Released under the MIT License.
 # Copyright, 2022, by Samuel Williams.
 
-eval_gemfile("../gems.rb")
+eval_gemfile("gems.rb")
 
 gem "rack", "~> 2.0"

gems/rack-v3.rb

@@ -3,6 +3,6 @@
 # Released under the MIT License.
 # Copyright, 2022, by Samuel Williams.
 
-eval_gemfile("../gems.rb")
+eval_gemfile("gems.rb")
 
 gem "rack", "~> 3.0"

lib/protocol/rack/adapter.rb

@@ -11,7 +11,7 @@
 module Protocol
 	module Rack
 		module Adapter
-			if ::Rack::RELEASE >= "3"
+			if ::Rack.release >= "3"
 				IMPLEMENTATION = Rack3
 			else
 				IMPLEMENTATION = Rack2

lib/protocol/rack/adapter/rack2.rb

@@ -108,10 +108,12 @@ def wrap_headers(fields)
 
 						if key.start_with?('rack.')
 							meta[key] = value
-						else
+						elsif value.is_a?(String)
 							value.split("\n").each do |value|
 								headers[key] = value
 							end
+						else
+							headers[key] = value
 						end
 					end
 

test/protocol/rack/adapter.rb

@@ -75,6 +75,21 @@
 		end
 	end
 
+	with 'non-string header value' do
+		let(:app) do
+			lambda do |env|
+				[200, {'x-custom' => 123}, ["Hello World!"]]
+			end
+		end
+		
+		let(:response) {client.get("/")}
+		
+		it "get valid response" do
+			expect(response.read).to be == "Hello World!"
+			expect(response.headers['x-custom']).to be == ['123']
+		end
+	end
+	
 	with 'REQUEST_URI', timeout: 1 do
 		let(:app) do
 			lambda do |env|