1637101 - [wpt-sync] Sync PR 23516 - Fix module script referrer port

Status: RESOLVED FIXED

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

Description

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

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

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

Dominic Farolino <[e-mail]> wrote:

Fix module script referrer port

The origin-when-cross-origin module script descendant test requires maintaining a variable remoteOrigin, whose value is the "cross-origin" origin that we're using to host remote-origin scripts for the test. It is initialized as:

const remoteOrigin = "http://{{domains[www1]}}:{{ports[http][0]}}/";

The problematic bit is ports[http][0]. When running this locally, it is not port 80, so tests such as this one report a referrer with a non-80 port, and the comparison works.

However on sites like wpt.live, this test seems to fail (in Chrome, whose implementation is spec-conformant, and obviously in any browser that isn't). It is because referrerOrigin contains the port number :80, but the generated Referer from the browser never contains the port number :80 if that's where the site is hosted. There are two solutions:

• Definitely use a non-80 port-number (which is what this PR does)
• Detect if {{ports[http][0]}} == 80, and strip the :80 from referrerOrigin
  • This seemed complex, so I didn't do it

Anne, do you know why browsers do not include :80 in the Referer? I couldn't find it from an admittedly cursory search through Fetch and Referrer Policy.

Comment 1

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=2cd4a01a4347a65e872ab3784b46f9eaa5c85075

Comment 2

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

CI Results

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

Total 1 tests and 6 subtests

Status Summary

Firefox

OK : 1
PASS: 4
FAIL: 2

Chrome

OK : 1
PASS: 6

Safari

OK : 1
PASS: 3
FAIL: 3

Links

GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

/html/semantics/scripting-1/the-script-element/module/referrer-origin-when-cross-origin.sub.html
Importing a remote-origin descendant script from a remote-origin top-level script with the origin-when-cross-origin policy.: FAIL (Chrome: PASS, Safari: FAIL)
Importing a same-origin descendant script from a remote-origin top-level script with the origin-when-cross-origin policy.: FAIL (Chrome: PASS, Safari: FAIL)

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/4bdd2bf8bfdc
[wpt PR 23516] - Fix module script referrer port, a=testonly

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/eb53db0f2b4e
[wpt PR 23516] - Fix module script referrer port, a=testonly

Comment 5

[Andreea Pavel [:apavel]]

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