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 descriptionChrome 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).
,
Dec 9 2016
,
Dec 28 2016
Upload CL for this issue https://codereview.chromium.org/2603713003/
,
Jan 7 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/0f762fedb8512772504cd5548b93f2523ec1e3a2 commit 0f762fedb8512772504cd5548b93f2523ec1e3a2 Author: corona10 <corona10@gmail.com> Date: Sat Jan 07 17:50:42 2017 [Fetch API] Pass web-platform fetch/api/cors/cors-preflight.html. 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 ", ". So I modify it from ", "(comma + space) to ","(comma). BUG= 672656 Review-Url: https://codereview.chromium.org/2603713003 Cr-Commit-Position: refs/heads/master@{#442170} [modify] https://crrev.com/0f762fedb8512772504cd5548b93f2523ec1e3a2/third_party/WebKit/LayoutTests/http/tests/fetch/script-tests/thorough/cors-preflight.js [modify] https://crrev.com/0f762fedb8512772504cd5548b93f2523ec1e3a2/third_party/WebKit/LayoutTests/http/tests/fetch/script-tests/thorough/cors-preflight2.js [modify] https://crrev.com/0f762fedb8512772504cd5548b93f2523ec1e3a2/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/resources/access-control-preflight-request-header-sorted.php [modify] https://crrev.com/0f762fedb8512772504cd5548b93f2523ec1e3a2/third_party/WebKit/Source/core/fetch/CrossOriginAccessControl.cpp [modify] https://crrev.com/0f762fedb8512772504cd5548b93f2523ec1e3a2/third_party/WebKit/Source/core/fetch/CrossOriginAccessControlTest.cpp
,
Jan 10 2017
Thanks corona10 |
|||
►
Sign in to add a comment |
|||
Comment 1 by tyoshino@chromium.org
, Dec 9 2016Summary: 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))