Bug 1021595 - Set "services.mobileid.forcehttps" to true once the server is https. r=jedp
authorFernando Jiménez <ferjmoreno@gmail.com>
Thu, 03 Jul 2014 11:56:34 +0200
changeset 192032 1f24cbde93deafacf3d7f3e7c7061f528dad2900
parent 192031 e257340ac8dc6c55b922b36190f4747d9a94369c
child 192033 96e880aaf1b7f10179e7665bbf2b261295e926af
push id8589
push userferjmoreno@gmail.com
push dateThu, 03 Jul 2014 09:57:00 +0000
treeherderb2g-inbound@1f24cbde93de [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjedp
bugs1021595
milestone33.0a1
Bug 1021595 - Set "services.mobileid.forcehttps" to true once the server is https. r=jedp
b2g/app/b2g.js
services/mobileid/MobileIdentityClient.jsm
--- a/b2g/app/b2g.js
+++ b/b2g/app/b2g.js
@@ -991,12 +991,13 @@ pref("touchcaret.enabled", false);
 pref("selectioncaret.enabled", false);
 
 // Enable sync and mozId with Firefox Accounts.
 #ifdef MOZ_SERVICES_FXACCOUNTS
 pref("services.sync.fxaccounts.enabled", true);
 pref("identity.fxaccounts.enabled", true);
 #endif
 
-pref("services.mobileid.server.uri", "http://msisdn.dev.mozaws.net");
+// Mobile Identity API.
+pref("services.mobileid.server.uri", "https://msisdn-dev.stage.mozaws.net");
 
 // Enable mapped array buffer
 pref("dom.mapped_arraybuffer.enabled", true);
--- a/services/mobileid/MobileIdentityClient.jsm
+++ b/services/mobileid/MobileIdentityClient.jsm
@@ -16,19 +16,18 @@ Cu.import("resource://services-common/ha
 Cu.import("resource://services-common/utils.js");
 Cu.import("resource://services-crypto/utils.js");
 Cu.import("resource://gre/modules/MobileIdentityCommon.jsm");
 Cu.import("resource://gre/modules/Promise.jsm");
 Cu.import("resource://gre/modules/Services.jsm");
 
 this.MobileIdentityClient = function(aServerUrl) {
   let serverUrl = aServerUrl || SERVER_URL;
-  let forceHttps = false;
+  let forceHttps = true;
   try {
-    // TODO: Force https in production. Bug 1021595.
     forceHttps = Services.prefs.getBoolPref(PREF_FORCE_HTTPS);
   } catch(e) {
     log.warn("Getting force HTTPS pref failed. If this was not intentional " +
              "check that " + PREF_FORCE_HTTPS + " is defined");
   }
 
   log.debug("Force HTTPS " + forceHttps);