kotovalexarian-likes-github/ruby--ruby
1
0
Fork 0
mirror of https://github.com/ruby/ruby.git synced 2022-11-09 12:17:21 -05:00
Code Releases Activity

raise error if value contains CR/LF in iniheader of initialize_http_header

Browse source 
like r59693, initialize_http_header also should raise error. [Bug #14208]

git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@61359 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
This commit is contained in:
naruse 2017-12-20 12:06:59 +00:00
parent 677c539d1f
commit 0078e40115
2 changed files with 22 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
lib/net/http/header.rb
View file

@ -18,7 +18,11 @@ module Net::HTTPHeader
if value.nil?
warn "net/http: nil HTTP header: #{key}", uplevel: 1 if $VERBOSE
else
@header[key.downcase] = [value.strip]
value = value.strip # raise error for invalid byte sequences
if value.count("\r\n") > 0
raise ArgumentError, 'header field value cannot include CR/LF'
end
@header[key.downcase] = [value]
end
end
end
@ -75,8 +79,8 @@ module Net::HTTPHeader
append_field_value(ary, val)
@header[key.downcase] = ary
else
val = val.to_s
if /[\r\n]/n.match?(val.b)
val = val.to_s # for compatibility use to_s instead of to_str
if val.b.count("\r\n") > 0
raise ArgumentError, 'header field value cannot include CR/LF'
end
@header[key.downcase] = [val]

15
test/net/http/test_httpheader.rb
View file

@ -16,6 +16,21 @@ class HTTPHeaderTest < Test::Unit::TestCase
@c = C.new
end
def test_initialize
@c.initialize_http_header("foo"=>"abc")
assert_equal "abc", @c["foo"]
@c.initialize_http_header("foo"=>"abc", "bar"=>"xyz")
assert_equal "xyz", @c["bar"]
@c.initialize_http_header([["foo", "abc"]])
assert_equal "abc", @c["foo"]
@c.initialize_http_header([["foo", "abc"], ["bar","xyz"]])
assert_equal "xyz", @c["bar"]
assert_raise(NoMethodError){ @c.initialize_http_header("foo"=>[]) }
assert_raise(ArgumentError){ @c.initialize_http_header("foo"=>"a\nb") }
assert_raise(ArgumentError){ @c.initialize_http_header("foo"=>"a\rb") }
assert_raise(ArgumentError){ @c.initialize_http_header("foo"=>"a\xff") }
end
def test_size
assert_equal 0, @c.size
@c['a'] = 'a'