iOS cookie parsing differs from defacto standards |
|||||
Issue descriptionBecause Chrome on iOS relies on the OS cookie line parser (in particular, NSHTTPCookie and its parsing method cookiesWithResponseHeaderFields, see https://developer.apple.com/library/mac/documentation/Cocoa/Reference/Foundation/Classes/NSHTTPCookie_Class/#//apple_ref/occ/clm/NSHTTPCookie/cookiesWithResponseHeaderFields:forURL:), we cannot currently control how cookies are parsed. NSHTTPCookie seems to be pretty much spec compliant, but that leads to some noticeable differences with the de facto standards of all the other UAs, such as with empty-key cookies. See http://inikulin.github.io/cookie-compat/#NAME0028 for an example of where iOS differs, similar to what Safari does. Chrome is aiming to match these de facto standards, as per issue 601786 , but we can't do that on iOS. Thus, for now, we are blocking out some tests on iOS, which would fail (such as CookieStoreTest::EmptyKeyTest), but if there comes a day where either NSHTTPCookie matches the de facto standard, or we do our own cookie parsing on iOS, we should re-enable these tests.
,
Aug 17 2016
FWIW, I'm not sure there's anything to do in the short-term. I meant for this to be a tracker bug so that we have it on our plate for a future where either we aren't depending on NSHTTPCookie or its behavior is changed.
,
Aug 17 2016
Joel, should we file a radar for this issue? If so, may I ask you to do this (because I'm not familiar with details of the problem)?.
,
Aug 17 2016
No, I don't think we should right now. Technically they're following the spec, and we're only changing Chrome's behavior to match what the majority of other UAs are doing. There's a chance we'll add this behavior to the spec at some point, in which case it makes sense to file a radar, but not right now.
,
Aug 18 2017
Is this still an issue?
,
Aug 24
Is this still an issue? Should we close this bug?
,
Aug 24
Still an issue. |
|||||
►
Sign in to add a comment |
|||||
Comment 1 by justincohen@chromium.org
, Aug 17 2016Labels: M-55
Owner: eugene...@chromium.org
Status: Assigned (was: Untriaged)