Allow non-string value in Headers#[]=.

Author: ioquatix

lib/protocol/http/headers.rb

@@ -198,7 +198,11 @@ def extract(keys)
 			# @parameter key [String] the header key.
 			# @parameter value [String] the header value to assign.
 			def add(key, value)
-				self[key] = value
+				if @indexed
+					merge_into(@indexed, key.downcase, value)
+				end
+				
+				@fields << [key, value]
 			end
 
 			# Set the specified header key to the specified value, replacing any existing header keys with the same name.
@@ -230,11 +234,8 @@ def merge(headers)
 			# @parameter key [String] The header key.
 			# @parameter value [String] The header value.
 			def []= key, value
-				if @indexed
-					merge_into(@indexed, key.downcase, value)
-				end
-				
-				@fields << [key, value]
+				# The value MUST be a string, so we convert it to a string to prevent errors later on.
+				self.add(key, value.to_s)
 			end
 
 			# The policy for various headers, including how they are merged and normalized.

test/protocol/http/headers.rb

@@ -167,18 +167,25 @@
 	end
 
 	with "#[]=" do
-		it "can add field" do
+		it "can add field with a String value" do
+			headers["Content-Length"] = "1"
+			
+			expect(headers.fields.last).to be == ["Content-Length", "1"]
+			expect(headers["content-length"]).to be == "1"
+		end
+		
+		it "can add field with an Integer value" do
 			headers["Content-Length"] = 1
 
-			expect(headers.fields.last).to be == ["Content-Length", 1]
-			expect(headers["content-length"]).to be == 1
+			expect(headers.fields.last).to be == ["Content-Length", "1"]
+			expect(headers["content-length"]).to be == "1"
 		end
 
 		it "can add field with indexed hash" do
 			expect(headers.to_h).not.to be(:empty?)
 
-			headers["Content-Length"] = 1
-			expect(headers["content-length"]).to be == 1
+			headers["Content-Length"] = "1"
+			expect(headers["content-length"]).to be == "1"
 		end
 	end
 
@@ -241,7 +248,7 @@
 		it "can merge content-length" do
 			headers.merge!("content-length" => 2)
 
-			expect(headers["content-length"]).to be == 2
+			expect(headers["content-length"]).to be == "2"
 		end
 	end