Closed Bug 1733165 Opened 3 years ago Closed 3 years ago

[wpt-sync] Sync PR 30979 - Fetch: add more header normalization tests

Categories

(Core :: DOM: Networking, task, P4)

task

Tracking

()

RESOLVED FIXED
95 Branch
Tracking Status
firefox95 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 30979 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/30979
Details from upstream follow.

Luca Casonato <hello@lcas.dev> wrote:

Fetch: add more header normalization tests

The existing tests did not account for chars like \f being trimmed
with String.prototype.trim, but not in header values.

Ref: https://github.com/denoland/deno/pull/12233

cc @andreubotella

Component: web-platform-tests → DOM: Networking
Product: Testing → Core

CI Results

Ran 11 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 4 tests and 3 subtests

Status Summary

Firefox

OK : 4
PASS : 12

Chrome

OK : 4
PASS : 12

Safari

OK : 3
PASS : 9
ERROR: 1

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/4209a34fe905 [wpt PR 30979] - Fetch: add more header normalization tests, a=testonly
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 95 Branch
You need to log in before you can comment on or make changes to this bug.