1827624 - Change Search SERP Telemetry tests to use https

Status: RESOLVED FIXED

(Firefox :: Search, task, P3)

Description

[Mark Banner (:standard8)]

The SERP telemetry tests currently use a mixture of http and https. Since all the search engines are https-only, I see no harm for switching these to https.

We can also then limit the SearchSERPTelemetry actors to https-only processes (building on bug 1824897).

Comment 1

[James Teow [:jteow]]

Some of the search telemetry tests load mock pages and in-page resources from http://mochi.test:8888/, so maybe we could transform them to instead use https://example.com/.

Comment 2

[Mark Banner (:standard8)]

(In reply to James Teow [:jteow] from comment #1)

Some of the search telemetry tests load mock pages and in-page resources from http://mochi.test:8888/, so maybe we could transform them to instead use https://example.com/.

Yeah, I think we might have to be careful with some of them, as we were also designing them with a different domain to the ad servers.

Comment 3

[Mark Banner (:standard8)]

Attachment: Bug 1827624 - Restrict search SERP telemetry actors to https only pages. r?jteow!

We only monitor telemetry from sites that are already https-only.

Depends on D175422

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/ca8860ca94c8
Restrict search SERP telemetry actors to https only pages. r=jteow

Comment 5

[Atila Butkovits]

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