Closed Bug 1641871 Opened 5 years ago Closed 5 years ago

[wpt-sync] Sync PR 23848 - [wptserve] Fix response.status encoding

Categories

(Testing :: web-platform-tests, task, P4)

Product:
Testing
Component:
web-platform-tests
Type:
task

Tracking

(firefox79 fixed)

Status:
RESOLVED FIXED
Milestone:
mozilla79
Tracking Flags:
Tracking Status
firefox79 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

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

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

Robert Ma <robertma@chromium.org> wrote:

[wptserve] Fix response.status encoding

Previously, if a bytes message b"foo" is assigned to response.status in
Python 3, we'd end up getting "b'foo'" writen to the status line. This
change fixes that and clarifies that the setter accepts both binary and
text strings.

The PR was not expected to affect any tests, but the try push wasn't a success. Check the try results for infrastructure issues
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/fe6cde34c66a [wpt PR 23848] - [wptserve] Fix response.status encoding, a=testonly

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

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