1098559 - Install Firefox 33.1.1 from hotfix

Status: RESOLVED FIXED

(Firefox :: General, defect)

Description

[Stephen A Pohl [:spohl]]

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

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

We should update the hotfix to upgrade to 33.1

Comment 1

[Stephen A Pohl [:spohl]]

Attachment: Patch

This mirrors the patch from bug 1072615.

Comment 2

[Tyler Downer [He/Him]]

We should actually use Firefox 33.1.1, which fixes a startup crash for users upgrading from Firefox 32 and below.

Comment 3

[Stephen A Pohl [:spohl]]

Sorry, where's 33.1.1? I just asked about this yesterday on #developers and no-one seemed to be aware of a 33.1.1...

Comment 4

[Tyler Downer [He/Him]]

I believe it got go to build today, it will be put live tomorrow AFAIK

Comment 5

[Stephen A Pohl [:spohl]]

*cries*

Comment 6

[Tyler Downer [He/Him]]

Bug 1021265 is the fix included in 33.1.1 BTW

Comment 7

[Matthias Versen [:Matti]]

Stephen: You should bookmark https://hg.mozilla.org/releases/mozilla-release/

Comment 8

[Robert Strong (they/them - no direct email)]

Comment on attachment 8522491 [details] [diff] [review]
Patch

Bah... needs to be updated for 31.1.1 :(

Comment 9

[Stephen A Pohl [:spohl]]

Attachment: Patch

Comment 10

[Stephen A Pohl [:spohl]]

Attachment: Respect app.update.url.override for commbank

Don't apply hotfix if app.update.url.override starts with https://update.commbiz.commbank.com.au (see bug 928173 comment 32).

Comment 11

[Robert Strong (they/them - no direct email)]

Comment on attachment 8524091 [details] [diff] [review]
Respect app.update.url.override for commbank

>From: Stephen Pohl <[e-mail]>
>
>Bug 1098559: Don't apply hotfix if app.update.url.override starts with https://update.commbiz.commbank.com.au (see bug 928173 comment 32). r=rstrong
>
>diff --git a/v20140527.01/resource/update.jsm b/v20140527.01/resource/update.jsm
>--- a/v20140527.01/resource/update.jsm
>+++ b/v20140527.01/resource/update.jsm
>@@ -1009,16 +1009,28 @@ UpgradeManager.prototype = {
>       }
> 
>       if (!Services.prefs.getBoolPref("app.update.auto")) {
>         this._log.warn("Not applicable - app.update.auto disabled");
>         return false;
>       }
>     } catch (e) {}
> 
>+    // Don't install if app.update.url.override starts with
>+    // https://update.commbiz.commbank.com.au (see bug 928173 comment 32).
>+    try {
>+      let overrideUrl = Services.prefs.getCharPref("app.update.url.override");
>+      if (overrideUrl.startsWith("https://update.commbiz.commbank.com.au")) {
>+        this._log.warn("Not applicable - app.update.url.override starts with " +
>+                       "https://update.commbiz.commbank.com.au (bug 928173 " +
>+                        "comment 32)");
nit: alignment

>+        return false;
>+      }
>+    } catch (e) {}
>+
>     this._log.warn("Hotfix applicable");
>     return true;
>   },
> 
>   /**
>    * Whether it appears we have upgraded to the version we were supposed to.
>    */
>   _isUpgraded: function () {

Comment 12

[Stephen A Pohl [:spohl]]

Attachment: Respect app.update.url.override for commbank

Good eye! :-) Fixed alignment, carrying over r+.

Thanks, Robert!

Comment 13

[Stephen A Pohl [:spohl]]

http://hg.mozilla.org/releases/firefox-hotfixes/rev/3766e8634f7e
http://hg.mozilla.org/releases/firefox-hotfixes/rev/c8078da55da2

Comment 14

[Stephen A Pohl [:spohl]]

Paul, just a heads up that we're planning to release a new hotfix with a workaround in place like you described in bug 928173 comment 40.

Comment 15

[Paul]

Thanks all. Much appreciated.

Comment 16

[Stephen A Pohl [:spohl]]

Comment on attachment 8524096 [details] [diff] [review]
Respect app.update.url.override for commbank

Review of attachment 8524096 [details] [diff] [review]:
-----------------------------------------------------------------

::: v20140527.01/resource/update.jsm
@@ +1017,5 @@
> +    // Don't install if app.update.url.override starts with
> +    // https://update.commbiz.commbank.com.au (see bug 928173 comment 32).
> +    try {
> +      let overrideUrl = Services.prefs.getCharPref("app.update.url.override");
> +      if (overrideUrl.startsWith("https://update.commbiz.commbank.com.au")) {

Argh. String.prototype.startsWith() is experimental and only implemented in FF17+[1]. Since we want FF 14.0.1 in particular to perform this check, we'll need to change this to indexOf(). Will post followup shortly.

[1] https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/startsWith

Comment 17

[Stephen A Pohl [:spohl]]

Attachment: Replace String.startsWith() with String.indexOf()

Comment 18

[Stephen A Pohl [:spohl]]

Attachment: Replace String.startsWith() with String.indexOf()

This was supposed to check for '== 0'.

Comment 19

[Stephen A Pohl [:spohl]]

http://hg.mozilla.org/releases/firefox-hotfixes/rev/56c7c4faa8b7

Comment 20

[Stephen A Pohl [:spohl]]

Somewhere along the way we started linking directly to the installer on ftp.mozilla.org. Any updates thereafter maintained this mistake. We should link to the bouncer instead.

Comment 21

[Stephen A Pohl [:spohl]]

:gps is handling this in bug 1107156.