Closed Bug 1554304 Opened 5 years ago Closed 5 years ago

[wpt-sync] Sync PR 17006 - CORB should block event-stream, gzip and x-www-form-urlencoded.

Categories

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

Product:
Core
Component:
DOM: Networking
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla69
Tracking Flags:
Tracking Status
firefox69 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync upstream][necko-triaged])

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

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

Lukasz Anforowicz <lukasza@chromium.org> wrote:

CORB should block event-stream, gzip and x-www-form-urlencoded.

This CL adds CORB coverage for:

  1. text/event-stream, application/x-www-form-urlencoded, based on the
    code review discussion in a previous CL here:
    https://chromium-review.googlesource.com/c/chromium/src/+/1604244/4/services/network/cross_origin_read_blocking.cc#227

  2. application/gzip, which wasn't mentioned explicitly in the CR
    discussion above, but which is ranked #212 in the spreadsheet
    mentioned in https://github.com/whatwg/fetch/issues/860#issuecomment-457330454
    and therefore probably should have been included in r659671 together
    with x-gzip (ranked #54) and zip (ranked #71).

Bug: 802836
Change-Id: I8c10f900110a2cb471437a19425bfd5e38aed2fe

Reviewed-on: https://chromium-review.googlesource.com/1628809
WPT-Export-Revision: 7f89ec37fb0bfc7ef87f997c659393be6f2d6156

PR 17006 applied with additional changes from upstream: fda5ab7104976319ac87925bcba11abf9e91dbee, 5b362a558c572f72022cf1e9b1a80fb6ded5c789
Component: web-platform-tests → DOM: Networking
Product: Testing → Core
Failed to get results from try push
Whiteboard: [wptsync downstream] → [wptsync downstream][necko-triaged]
Pushed by james@hoppipolla.co.uk:
https://hg.mozilla.org/integration/mozilla-inbound/rev/2c70fb58e726
[wpt PR 17006] - CORB should block event-stream, gzip and x-www-form-urlencoded., a=testonly

https://hg.mozilla.org/mozilla-central/rev/2c70fb58e726

Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox69: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla69
Whiteboard: [wptsync downstream][necko-triaged] → [wptsync upstream][necko-triaged]
You need to log in before you can comment on or make changes to this bug.