Closed Bug 1553024 Opened 5 years ago Closed 5 years ago

[wpt-sync] Sync PR 16929 - Fix accidental origin header removal in redirects

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 16929 into mozilla-central (this bug is closed when the sync is complete).

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

Yutaka Hirano <yhirano@chromium.org> wrote:

Fix accidental origin header removal in redirects

net/ implementation removes an origin when it sees a redirect which
changes request's method. This break CORS when the CORS flag is set -
because in that case we need an origin header.

This CL fixes it.

Bug: 965018
Change-Id: I3137d3b781c0cb293fbb282901b8bb52b3c13651

Reviewed-on: https://chromium-review.googlesource.com/1621419
WPT-Export-Revision: 80348bd9bca3e4ecf29ff7a0b5337de014dea84a

Component: web-platform-tests → DOM: Networking
Product: Testing → Core
Whiteboard: [wptsync downstream] → [wptsync downstream error]
Whiteboard: [wptsync downstream error] → [wptsync downstream]
Whiteboard: [wptsync downstream] → [wptsync downstream][necko-triaged]
Whiteboard: [wptsync downstream][necko-triaged] → [wptsync downstream error]
Whiteboard: [wptsync downstream error] → [wptsync downstream]
Whiteboard: [wptsync downstream] → [wptsync downstream][necko-triaged]
Failed to get results from try push
Pushed by james@hoppipolla.co.uk:
https://hg.mozilla.org/integration/mozilla-inbound/rev/d943dabe2545
[wpt PR 16929] - Fix accidental origin header removal in redirects, a=testonly

https://hg.mozilla.org/mozilla-central/rev/d943dabe2545

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.