Headers values are not combined correctly |
||||||
Issue descriptionhttp://jsbin.com/tiyuxo/edit?js,console Appending headers of the same name should have their values joined by ','. Firefox also gets this wrong, but Safari gets it right(ish).
,
Dec 1 2016
See also: issue 670439
,
Dec 2 2016
blink::Headers::get() must be updated. Headers.get()'s semantics have been changed on this commit in Jan. https://github.com/whatwg/fetch/issues/189 https://github.com/whatwg/fetch/commit/42464c8c
,
Dec 2 2016
,
Dec 7 2016
bug 645492 (deprecation of getAll()) is blocked on this.
,
Dec 7 2016
,
Dec 17 2016
On going CL for this issue. https://codereview.chromium.org/2587463002/
,
Dec 20 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/7ad32cbf574e3d263b64f1888520b9158798d41b commit 7ad32cbf574e3d263b64f1888520b9158798d41b Author: corona10 <corona10@gmail.com> Date: Tue Dec 20 07:55:46 2016 Appending headers of the same name should have their values joined by ','. Now, FetchHeaderList::get() is iterating all header list and append strings to a result joined by ','. BUG= 645497 Review-Url: https://codereview.chromium.org/2587463002 Cr-Commit-Position: refs/heads/master@{#439743} [modify] https://crrev.com/7ad32cbf574e3d263b64f1888520b9158798d41b/third_party/WebKit/LayoutTests/http/tests/fetch/script-tests/headers.js [modify] https://crrev.com/7ad32cbf574e3d263b64f1888520b9158798d41b/third_party/WebKit/Source/modules/fetch/FetchHeaderList.cpp [modify] https://crrev.com/7ad32cbf574e3d263b64f1888520b9158798d41b/third_party/WebKit/Source/modules/fetch/Headers.cpp
,
Mar 28 2017
Fixed by ad32cbf574e3d263b64f1888520b9158798d41b.
,
Apr 20 2017
|
||||||
►
Sign in to add a comment |
||||||
Comment 1 by jakearchibald@chromium.org
, Sep 9 2016