Closed Bug 1816145 Opened 2 years ago Closed 2 years ago

[wpt-sync] Sync PR 38451 - Except where impractical, set Connection: Close with close_connection

Categories

(Core :: DOM: Networking, task, P4)

Product:
Core
Component:
DOM: Networking
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
112 Branch
Tracking Flags:
Tracking Status
firefox112 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

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

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

Sam Sneddon <gsnedders@apple.com> wrote:

Except where impractical, set Connection: Close with close_connection

Because of the unexpected close, CFNetwork sometimes runs into a retry limit trying to find a working connection. Given we SHOULD be closing gracefully, and these tests are not attempting to test abrupt closure, it makes sense to just change these tests.

The files not changed here are:

tools/wptserve/wptserve/server.py
tools/wptserve/wptserve/handlers.py
tools/wptserve/wptserve/response.py
fetch/h1-parsing/resources/status-code.py
fetch/h1-parsing/resources/message.py

See also https://github.com/web-platform-tests/wpt/pull/38091.

Component: web-platform-tests → DOM: Networking
Product: Testing → Core
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/864b2d75730a [wpt PR 38451] - Except where impractical, set Connection: Close with close_connection, a=testonly
Test result changes from PR not available.
Test result changes from PR not available.
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/ab9039d12af9 [wpt PR 38451] - Except where impractical, set Connection: Close with close_connection, a=testonly

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

Status: NEW → RESOLVED
Closed: 2 years ago
status-firefox112: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 112 Branch
Test result changes from PR not available.
You need to log in before you can comment on or make changes to this bug.