Closed
Bug 1515298
Opened 5 years ago
Closed 5 years ago
[wpt-sync] Sync PR 14599 - SignedExchange: Reject SXG if fallback URL has invalid UTF-8 sequence
Categories
(Testing :: web-platform-tests, enhancement, P4)
Testing
web-platform-tests
Tracking
(firefox67 fixed)
RESOLVED
FIXED
mozilla67
Tracking | Status | |
---|---|---|
firefox67 | --- | fixed |
People
(Reporter: mozilla.org, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 14599 into mozilla-central (this bug is closed when the sync is complete). PR: https://github.com/web-platform-tests/wpt/pull/14599 Details from upstream follow. Kunihiko Sakamoto <ksakamoto@chromium.org> wrote: > SignedExchange: Reject SXG if fallback URL has invalid UTF-8 sequence > > GURL parser happily accepts invalid UTF-8 path, so we need to validate > the fallback URL string before parsing. > > Spec: https://github.com/WICG/webpackage/pull/346 > > Bug: 916390 > Change-Id: Ife25621e2a41beef01cbf36a5ab523eaee1ea222 > > Reviewed-on: https://chromium-review.googlesource.com/1382724 > WPT-Export-Revision: 59700b0cc6ed35582fe74e3466b553c48fd9164a
Assignee | ||
Comment 1•5 years ago
|
||
PR 14599 applied with additional changes from upstream: 2e19cbe5d5d74821056d4ae2cef435accefaf4ef
Assignee | ||
Comment 2•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=7ea58df0a2727711c4223a86dedc2909c3a1eb3d
Assignee | ||
Comment 3•5 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=04d6fef0d56e0fb6f506198374bad259fcb88eff
Assignee | ||
Comment 4•5 years ago
|
||
Ran 1 tests and 1 subtests OK : 1 PASS : 1
Pushed by james@hoppipolla.co.uk: https://hg.mozilla.org/integration/mozilla-inbound/rev/812e25f08167 [wpt PR 14599] - SignedExchange: Reject SXG if fallback URL has invalid UTF-8 sequence, a=testonly
Pushed by james@hoppipolla.co.uk: https://hg.mozilla.org/integration/mozilla-inbound/rev/c680ff16a9c7 [wpt PR 14599] - SignedExchange: Reject SXG if fallback URL has invalid UTF-8 sequence, a=testonly
Comment 7•5 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox67:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
You need to log in
before you can comment on or make changes to this bug.
Description
•