848263 - Use HTTPS instead of HTTP for in-product support.mozilla.org links for metro

Status: RESOLVED FIXED

(Firefox for Metro Graveyard :: General, defect)

Description

[Raymond Lee [:raymondlee]]

+++ This bug was initially created as a clone of Bug #840687 +++
+++ This bug was initially created as a clone of Bug #771788 +++

app.support.baseURL;http://support.mozilla.org/1/firefox/%VERSION%/%OS%/%LOCALE%/

Should become https:// at some point.

(In reply to James Socol [:jsocol, :james] from comment #2)
> (In reply to :Gavin Sharp (use [e-mail] for email) from comment
> #1)
> > These bugs are probably best filed per-host. support.mozilla.org link
> > changes require sign-off from different people than mozilla.com links. I
> > think mozilla.com/org can probably just be changed wholesale.
> 
> support.mozilla.org in particular is not really designed to have all traffic
> come over HTTPS right now. I don't think it would cause the site to fall
> over, but there are changes we would really rather make before Firefox
> changed to HTTPS-only.

Comment 1

[Matt Brubeck (:mbrubeck)]

I'm happy to review a patch for this, but we won't be able to land the patch until the SUMO team is ready (bug 848520).

This is a good trivial bug for someone getting started in Firefox development who wants to contribute a patch.  It's just a change to the pref value in this file:
http://mxr.mozilla.org/mozilla-central/source/browser/metro/profile/metro.js

Comment 2

[Ekanan Ketunuti]

Attachment: proposed patch

Comment 3

[Matt Brubeck (:mbrubeck)]

Comment on attachment 722640 [details] [diff] [review]
proposed patch

Wow, that was quick.  Thanks!

As mentioned above, this patch can land once the support.mozilla.org team is ready for it.

Comment 4

[Ricky Rosario [:rrosario, :r1cky]]

We are all set from the SUMO side here. We've been redirecting all the in-product links to https for a few weeks now and haven't had any issues.

Comment 5

[Ekanan Ketunuti]

Attachment: Unbitrotted patch

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/ca8a60fff495

Comment 7

[Ryan VanderMeulen [:RyanVM]]

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