Closed Bug 1761201 Opened 2 years ago Closed 2 years ago

[wpt-sync] Sync PR 33341 - Add a WPT for 304 and CORP interaction.

Categories

(Core :: DOM: Core & HTML, task, P4)

Product:
Core
Component:
DOM: Core & HTML
Type:
task

Tracking

()

Status:
RESOLVED INVALID

People

(Reporter: mozilla.org, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

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

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

Arthur Hemery <ahemery@chromium.org> wrote:

Add a WPT for 304 and CORP interaction.

Currently, 304 responses do not set the CORP header, which seems to be
the correct behavior according to
https://datatracker.ietf.org/doc/html/rfc7232#section-4.1

On the other hand this causes issue because we incorrectly consider
that the cached request did not set CORP, and ends up being blocked by
COEP, which is definitely not an appropriate behavior.

This introduces a WPT verifying the behavior, and does some minor
cosmetic changes to the test it's based upon.

Bug: 1241264
Change-Id: Id44439cdde63b292377587b74bdd9b6fa10f1e65
Reviewed-on: https://chromium-review.googlesource.com/3547677
WPT-Export-Revision: 7de4862ecea77ef2401ee3692ea74f1412b98d20

Component: web-platform-tests → DOM: Core & HTML
Product: Testing → Core
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → INVALID
Whiteboard: [wptsync downstream] → [wptsync downstream error]
Whiteboard: [wptsync downstream error] → [wptsync downstream]

CI Results

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

Total 2 tests and 5 subtests

Status Summary

Safari

OK : 2
PASS: 9
FAIL: 1

Links

GitHub PR Head
GitHub PR Base

Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/9cd715b4db65
[wpt PR 33341] - Add a WPT for 304 and CORP interaction., a=testonly
You need to log in before you can comment on or make changes to this bug.