diff options
-rw-r--r-- | ChangeLog | 10 | ||||
-rw-r--r-- | lib/webrick/cookie.rb | 2 | ||||
-rw-r--r-- | test/webrick/test_cookie.rb | 20 |
3 files changed, 31 insertions, 1 deletions
@@ -1,3 +1,13 @@ +Fri Apr 1 18:53:06 2011 NAKAMURA, Hiroshi <nahi@ruby-lang.org> + + * lib/webrick/cookie.rb (WEBrick::Cookie.parse): 'white space is + permitted between tokens' according to RFC2965. Though 'Netscape + spec' does not define the syntax clearly, make it tolerant as a + server. As a real-world example, rest-client gem sends + 'Cookie: foo=1;bar=2' + + * test/webrick/test_cookie.rb (test_parse_non_whitespace): test it. + Fri Apr 1 13:19:20 2011 Nobuyoshi Nakada <nobu@ruby-lang.org> * vm_core.h (RUBY_VM_CHECK_INTS_TH): merge a patch by ko1 diff --git a/lib/webrick/cookie.rb b/lib/webrick/cookie.rb index 814e6645a3..eb3b63b006 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 14771fd01c..1275dc8eff 100644 --- a/test/webrick/test_cookie.rb +++ b/test/webrick/test_cookie.rb @@ -54,6 +54,26 @@ class TestWEBrickCookie < Test::Unit::TestCase assert_equal("9865ecfd514be7f7", cookies[1].value) end + def test_parse_non_whitespace + data = [ + '$Version="1";', + 'Customer="WILE_E_COYOTE";$Path="/acme";', + 'Part_Number="Rocket_Launcher_0001";$Path="/acme";', + '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) + end + def test_parse_set_cookie data = %(Customer="WILE_E_COYOTE"; Version="1"; Path="/acme") cookie = WEBrick::Cookie.parse_set_cookie(data) |