mirror of
https://github.com/moparisthebest/curl
synced 2024-11-10 19:45:04 -05:00
7f7fcd0d75
RFC 6265 section 4.1.1 spells out that the first name/value pair in the header is the actual cookie name and content, while the following are the parameters. libcurl previously had a more liberal approach which causes significant problems when introducing new cookie parameters, like the suggested new cookie priority draft. The previous logic read all n/v pairs from left-to-right and the first name used that wassn't a known parameter name would be used as the cookie name, thus accepting "Set-Cookie: Max-Age=2; person=daniel" to be a cookie named 'person' while an RFC 6265 compliant parser should consider that to be a cookie named 'Max-Age' with an (unknown) parameter 'person'. Fixes #709
59 lines
1012 B
Plaintext
59 lines
1012 B
Plaintext
<testcase>
|
|
<info>
|
|
<keywords>
|
|
HTTP
|
|
HTTP GET
|
|
HTTP proxy
|
|
cookies
|
|
</keywords>
|
|
</info>
|
|
|
|
# This test is very similar to 1216, only that it sets the cookies from the
|
|
# first site instead of reading from a file
|
|
<reply>
|
|
<data>
|
|
HTTP/1.1 200 OK
|
|
Date: Tue, 25 Sep 2001 19:37:44 GMT
|
|
Set-Cookie: bug=fixed; domain=.example.fake;
|
|
Content-Length: 21
|
|
|
|
This server says moo
|
|
</data>
|
|
</reply>
|
|
|
|
# Client-side
|
|
<client>
|
|
<server>
|
|
http
|
|
</server>
|
|
<name>
|
|
HTTP cookies and domains with same prefix
|
|
</name>
|
|
<command>
|
|
http://example.fake/c/1218 http://example.fake/c/1218 http://bexample.fake/c/1218 -b nonexisting -x %HOSTIP:%HTTPPORT
|
|
</command>
|
|
</client>
|
|
|
|
# Verify data after the test has been "shot"
|
|
<verify>
|
|
<strip>
|
|
^User-Agent:.*
|
|
</strip>
|
|
<protocol>
|
|
GET http://example.fake/c/1218 HTTP/1.1
|
|
Host: example.fake
|
|
Accept: */*
|
|
|
|
GET http://example.fake/c/1218 HTTP/1.1
|
|
Host: example.fake
|
|
Accept: */*
|
|
Cookie: bug=fixed
|
|
|
|
GET http://bexample.fake/c/1218 HTTP/1.1
|
|
Host: bexample.fake
|
|
Accept: */*
|
|
|
|
</protocol>
|
|
</verify>
|
|
</testcase>
|