1744924 - adjust more tests to work with https-first mode

Status: RESOLVED FIXED

(Core :: DOM: Security, task, P3)

Description

[Frederik Braun [:freddy]]

++ This bug was filed as a friend of bug 1743626 ++

We need to repair some tests so they don't break due to http->https upgrades e.g., when sites suddenly become same/cross-origin unexpectedly.

Comment 1

[Frederik Braun [:freddy]]

Attachment: Bug 1744924 - Run mixed-content tests with https-first mode disabled r?pbz

Comment 2

[Frederik Braun [:freddy]]

Let's leave this bug open in case there are more tests that need annotating.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/fb84c99d9d03
Run mixed-content tests with https-first mode disabled r=ckerschb

Comment 4

[Cristian Tuns]

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

Comment 5

[Frederik Braun [:freddy]]

Attachment: Bug 1744924 - Part 2: Adjust iframe sandbox test r?ckerschb

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/ba9a1fa63341
Part 2: Adjust iframe sandbox test r=ckerschb

Comment 7

[Atila Butkovits]

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

Comment 8

[Atila Butkovits]

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

Comment 9

[Frederik Braun [:freddy]]

Attachment: Bug 1744924 - adjust latest test for https-first mode r?ckerschb

This adjust some recently added/modified tests to compyl with
https-first enabled.

1. browser_ext_webRequest_error_after_stopped_or_closed.js
   The test opens a new tab for top navigation and expects it to stay HTTP.
   The test was modified to open-as HTTPS and expect as HTTPS.

2. browser_net_har_copy_all_as_har.js
   The test performs a request to an HTTP resource.
   The test was modified to open-as HTTPS and expect as HTTPS, furthermore
   the test had more expectation about the outgoing request and its
   response, e..g, the amount of headers. Those things had to be modified,
   as a secure connection also inludes Sec-Fetch-* headers and
   Upgrade-Insecure-Requests

3. browser_download_idn_blocklist.js
   The server runs its own webserver, which was non-trivial to switch to
   HTTPS, so we are now running this test with https-first mode always
   disabled.

Comment 10

[BugBot [:suhaib / :marco/ :calixte]]

The leave-open keyword is there and there is no activity for 6 months.
:freddy, maybe it's time to close this bug?
For more information, please visit auto_nag documentation.