diff options
-rw-r--r-- | ChangeLog | 10 | ||||
-rw-r--r-- | lib/webrick/cookie.rb | 2 | ||||
-rw-r--r-- | test/webrick/test_cookie.rb | 35 |
3 files changed, 32 insertions, 15 deletions
@@ -1,3 +1,13 @@ +Mon Jun 20 15:41:33 2011 Hiroshi Nakamura <nahi@ruby-lang.org> + + * lib/webrick/cookie.rb (WEBrick::Cookie.parse): Revert r31228. + r31228 was for allowing the 'Cookie:' header which did not have no + SP after ';' for separating cookie-pairs but RFC6265 requires single + SP after ';' there. We allow multiple SPs here for compatibility + with older WEBrick version. + + * test/webrick/test_cookie.rb: Test it. + Sun Jun 19 13:31:26 2011 Shota Fukumori <sorah@tubusu.net> * NEWS: Introduce --hide-skip on test/unit. diff --git a/lib/webrick/cookie.rb b/lib/webrick/cookie.rb index eb3b63b006..814e6645a3 100644 --- a/lib/webrick/cookie.rb +++ b/lib/webrick/cookie.rb @@ -57,7 +57,7 @@ module WEBrick ret = [] cookie = nil ver = 0 - str.split(/[;,]\s*/).each{|x| + str.split(/[;,]\s+/).each{|x| key, val = x.split(/=/,2) val = val ? HTTPUtils::dequote(val) : "" case key diff --git a/test/webrick/test_cookie.rb b/test/webrick/test_cookie.rb index 48f0235aeb..1652f39ee0 100644 --- a/test/webrick/test_cookie.rb +++ b/test/webrick/test_cookie.rb @@ -34,6 +34,7 @@ class TestWEBrickCookie < Test::Unit::TestCase data << 'Part_Number="Rocket_Launcher_0001"; $Path="/acme"; ' data << 'Shipping="FedEx"; $Path="/acme"' cookies = WEBrick::Cookie.parse(data) + assert_equal(3, cookies.size) assert_equal(1, cookies[0].version) assert_equal("Customer", cookies[0].name) assert_equal("WILE_E_COYOTE", cookies[0].value) @@ -54,24 +55,30 @@ class TestWEBrickCookie < Test::Unit::TestCase assert_equal("9865ecfd514be7f7", cookies[1].value) end - def test_parse_non_whitespace + def test_parse_no_whitespace data = [ - '$Version="1";', - 'Customer="WILE_E_COYOTE";$Path="/acme";', - 'Part_Number="Rocket_Launcher_0001";$Path="/acme";', + '$Version="1"; ', + 'Customer="WILE_E_COYOTE";$Path="/acme";', # no SP between cookie-string + 'Part_Number="Rocket_Launcher_0001";$Path="/acme";', # no SP between cookie-string 'Shipping="FedEx";$Path="/acme"' ].join cookies = WEBrick::Cookie.parse(data) - assert_equal(1, cookies[0].version) - assert_equal("Customer", cookies[0].name) - assert_equal("WILE_E_COYOTE", cookies[0].value) - assert_equal("/acme", cookies[0].path) - assert_equal(1, cookies[1].version) - assert_equal("Part_Number", cookies[1].name) - assert_equal("Rocket_Launcher_0001", cookies[1].value) - assert_equal(1, cookies[2].version) - assert_equal("Shipping", cookies[2].name) - assert_equal("FedEx", cookies[2].value) + assert_equal(1, cookies.size) + end + + def test_parse_too_much_whitespaces + # According to RFC6265, + # cookie-string = cookie-pair *( ";" SP cookie-pair ) + # So single 0x20 is needed after ';'. We allow multiple spaces here for + # compatibility with older WEBrick versions. + data = [ + '$Version="1"; ', + 'Customer="WILE_E_COYOTE";$Path="/acme"; ', # no SP between cookie-string + 'Part_Number="Rocket_Launcher_0001";$Path="/acme"; ', # no SP between cookie-string + 'Shipping="FedEx";$Path="/acme"' + ].join + cookies = WEBrick::Cookie.parse(data) + assert_equal(3, cookies.size) end def test_parse_set_cookie |