Closed Bug 1545574 Opened 6 years ago Closed 6 years ago

[wpt-sync] Sync PR 16319 - [xhr] Remove None headers from the server handler

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: wpt-sync, Unassigned)

References

(
URL
)

Details

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

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

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

Robert Ma <robertma@chromium.org> wrote:

[xhr] Remove None headers from the server handler

authentication.py incorrectly returns some headers with None as the
value, which causes exceptions in wptserve.

The file was created in 234a672d023e8f57f3c78ca129788ca650242cc0 as a
rewrite from some PHP code and has been buggy since. This commit fixes
the bug by making the Python code more faithful to the original
XMLHttpRequest/resources/auth1/auth.php.

Whiteboard: [wptsync downstream] → [wptsync downstream error]
Component: web-platform-tests → DOM: Networking
Product: Testing → Core
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/9919a3d10965 [wpt PR 16319] - [xhr] Remove None headers from the server handler, a=testonly

https://hg.mozilla.org/mozilla-central/rev/9919a3d10965

Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox69: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla69
You need to log in before you can comment on or make changes to this bug.