Mirrors/servo
1
0
Fork 0
mirror of https://github.com/servo/servo.git synced 2025-08-05 21:50:18 +01:00
Code Issues Projects Releases Packages Wiki Activity

Auto merge of #26438 - Eijebong:headers, r=jdm

Browse source 
Separate values in Headers with ", ", not ","

Tests expect header values to be separated by that so this fixes a few
of them.
This commit is contained in:
bors-servo 2020-05-05 18:15:00 -04:00 • committed by GitHub
commit 81b92b9c1a
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 1 additions and 15 deletions
Download patch file Download diff file Expand all files Collapse all files

2
components/script/dom/headers.rs
View file

@ -96,7 +96,7 @@ impl HeadersMethods for Headers {
.get(HeaderName::from_str(&valid_name).unwrap())
{
combined_value = v.as_bytes().to_vec();
combined_value.push(b',');
combined_value.extend(b", ");
}
combined_value.extend(valid_value.iter().cloned());
match HeaderValue::from_bytes(&combined_value) {

14
tests/wpt/metadata/fetch/api/headers/headers-combine.html.ini
View file

@ -1,14 +0,0 @@
[headers-combine.html]
type: testharness
[Create headers using same name for different values]
expected: FAIL
[Check append methods when called with already used name]
expected: FAIL
[Iterate combined values]
expected: FAIL
[Iterate combined values in sorted order]
expected: FAIL