1109475 - Firefox should use HTTPS instead of HTTP for Safe Browsing URLs

Status: VERIFIED FIXED

(Toolkit :: Safe Browsing, defect)

Description

[Chris Peterson [:cpeterson]]

The current Safe Browsing URL prefs don't use HTTPS:

pref("browser.safebrowsing.reportGenericURL", "http://%LOCALE%.phish-generic.mozilla.com/?hl=%LOCALE%");
pref("browser.safebrowsing.reportErrorURL", "http://%LOCALE%.phish-error.mozilla.com/?hl=%LOCALE%");
pref("browser.safebrowsing.reportPhishURL", "http://%LOCALE%.phish-report.mozilla.com/?hl=%LOCALE%");
pref("browser.safebrowsing.reportMalwareURL", "http://%LOCALE%.malware-report.mozilla.com/?hl=%LOCALE%");
pref("browser.safebrowsing.reportMalwareErrorURL", "http://%LOCALE%.malware-error.mozilla.com/?hl=%LOCALE%");

https://mxr.mozilla.org/mozilla-central/source/browser/app/profile/firefox.js#1011

https://mxr.mozilla.org/mozilla-central/source/mobile/android/app/mobile.js#596

Comment 1

[Justin Dolske [:Dolske]]

Looks like we redirect to Google, and that's already supporting SSL, so this should just be a simple fix.

Although it also looks like a bunch of these prefs are not actually used anywhere (even indirectly, via getReportURL), so I'm got to take a stab at cleaning that up too.

Comment 2

[Justin Dolske [:Dolske]]

Attachment: Patch v.1

Haven't actually tested this yet, but should work! *crosses fingers*

Also, I kinda wanted to give browser.safebrowsing.malware.reportURL a similar cleanup, but that's a yak too far. It's used slightly differently in the code, and I'm already straying from the core purpose of this bug.

Comment 3

[Justin Dolske [:Dolske]]

Need bug 1138797 fixed before this can land, NI myself so it stays on my radar.

Comment 4

[Justin Dolske [:Dolske]]

Attachment: Patch v.1 (updated for bitrot)

Updated to apply cleanly to current mozilla-central.

Comment 5

[Justin Dolske [:Dolske]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=78e4713c9d92

Comment 6

[Justin Dolske [:Dolske]]

Attachment: Patch v.2

Oops. When I was finalizing the original patch I renamed the prefs, but didn't change the code that used those prefs. >_<

Verified that this all works now, and the server changes in bug 1138797 are live.

I created a small testplan in that bug (attachment 8622778 [details]), that might be useful for future checking.

Comment 7

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/1a21a5d0e9da

Comment 8

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/1a21a5d0e9da

Comment 9

[[github robot]]

Commit pushed to master at https://github.com/mozilla/addon-sdk

https://github.com/mozilla/addon-sdk/commit/96ae8d914fab9baad903cac07bf9f37da98fc0bc
Bug 1109475 - Firefox should use HTTPS instead of HTTP for Safe Browsing URLs. r=gcp

Comment 10

[Camelia Badau [:cbadau], Desktop QA]

Verified fixed on latest Aurora 41.0a2 (buildID: 20150730004009).