New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 672656 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner: ----
Closed: Jan 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Access-Control-Request-Headers has header names separated by comma+space, but should be separated by comma only (no space)

Reported by sideshowbarker@gmail.com, Dec 9 2016

Issue description

Chrome Version       : 57.0.2946.0 Revision 883e7b724370402ca8709a69ccec538e04f0bc2b-refs/heads/master@{#437239}
URLs : https://github.com/w3c/web-platform-tests/blob/master/fetch/api/cors/cors-preflight.html http://web-platform.test:8000/fetch/api/cors/cors-preflight.html
Other browsers tested:
  Add OK or FAIL, along with the version, after other browsers where you
have tested this issue:
     Safari: OK (WebKit Nightly)
    Firefox: OK
         IE: Fail [no fetch() support]

What steps will reproduce the problem?
(1) Run https://github.com/w3c/web-platform-tests/blob/master/fetch/api/cors/cors-preflight.html in a browser (e.g., from http://web-platform.test:8000/fetch/api/cors/cors-preflight.html, if you have a local web-platform-tests environment set up)
(2) Notice failures for “assert_equals: Access-Control-Allow-Headers value expected … but got …”

What is the expected result?
Expected no failures.


What happens instead?
Got failures for “assert_equals: Access-Control-Allow-Headers value expected … but got …”

Please provide any additional information below. Attach a screenshot if
possible.

The failures are because the expected result is a list of header names separated only be "," (comma), but Blink instead returns a list of header names separate by ", " (comma+space).

 
Status: Available (was: Unconfirmed)
Summary: Access-Control-Request-Headers has header names separated by comma+space, but should be separated by comma only (no space) (was: Access-Control-Allow-Headers has header names separated by comma+space, but should be separated by comma only (no space))
Thanks Michael.

Adjusted the title. The test is checking the value of Access-Control-Request-Headers sent by the UA echoed by preflight.py.

FTR, the spec side change was made in this issue:
https://github.com/whatwg/fetch/issues/249
https://github.com/whatwg/fetch/commit/1cc34b19e0bca5657a13a7ea96b73cd7fed1d8a8
Components: Blink>Network>XHR

Comment 3 by coron...@gmail.com, Dec 28 2016

Upload CL for this issue
https://codereview.chromium.org/2603713003/
Project Member

Comment 4 by bugdroid1@chromium.org, Jan 7 2017

Labels: Hotlist-Interop
Status: Fixed (was: Available)
Thanks corona10

Sign in to add a comment