666958 - Firefox 3.6.18 upgrade box displays release notes page instead of details page

Status: RESOLVED FIXED

(Release Engineering :: General, defect, P2)

Description

[Pascal Chevrel:pascalc (PTO until April 26)]

Attachment: Screenshot of the bug

(Tonnes our Dutch localizer reported this bug)

On windows 2000/XP, whatever be the locale (tested nl, fr, en-US), if you go to the Help menu and click on Check for updates, the update box loads Firefox 5 release notes instead of the details page, that is:

http://www.mozilla.com/en-US/firefox/5.0/releasenotes/
instead of presumably
http://www.mozilla.com/en-US/firefox/3.6/details/

See the screenshot for the problem.

I am tentatively filing that as a mozilla.com bug although I couldn't find a redirect in our site that would cause that bug, maybe the url hardcoded in Firefox 3.6.18 update mechanism is incorrect and this bug should moved to products.

Comment 1

[Rimas Kudelis [use needinfo to ping]]

Actually, the problem is in the XML file returned by the update server:

https://aus2.mozilla.org/update/3/Firefox/3.6.18/20110614230723/WINNT_x86-msvc/en-US/release/Windows_NT%205.1/default/default/update.xml?force=1

Not sure which product/component this warrants though...

Comment 2

[Pascal Chevrel:pascalc (PTO until April 26)]

moving to AUS/general, I think that's a better component

Comment 3

[Nick Thomas [:nthomas] (UTC+12)]

detailsURL is set to 
  https://www.mozilla.com/%LOCALE%/firefox/5.0/releasenotes/
where normally we'd set it to 
  https://www.mozilla.com/%LOCALE/firefox/5.0/details/
for a major update. However that doesn't exist, and in fact just got filed today in bug 668108.

Over to rail/bheasrum to untangle after Christian tells us what he wants to do.

Comment 4

[[e-mail] (:bhearsum)]

(In reply to comment #3)
> detailsURL is set to 
>   https://www.mozilla.com/%LOCALE%/firefox/5.0/releasenotes/
> where normally we'd set it to 
>   https://www.mozilla.com/%LOCALE/firefox/5.0/details/
> for a major update. However that doesn't exist, and in fact just got filed
> today in bug 668108.
> 
> Over to rail/bheasrum to untangle after Christian tells us what he wants to
> do.

Christian, what say you?

Comment 5

[Al Billings [:abillings - ex-MoCo]]

Five days later... Is this still unresolved?

Comment 6

[LegNeato]

(In reply to comment #5)
> Five days later... Is this still unresolved?

Yes.

(In reply to comment #4)
> (In reply to comment #3)
> > detailsURL is set to 
> >   https://www.mozilla.com/%LOCALE%/firefox/5.0/releasenotes/
> > where normally we'd set it to 
> >   https://www.mozilla.com/%LOCALE/firefox/5.0/details/
> > for a major update. However that doesn't exist, and in fact just got filed
> > today in bug 668108.
> > 
> > Over to rail/bheasrum to untangle after Christian tells us what he wants to
> > do.
> 
> Christian, what say you?

Point it to the details link. I've pushed a stopgap there for en-US (hopefully localization will follow shortly). Please check with me before putting the new snippets on the wire.

Comment 7

[Rimas Kudelis [use needinfo to ping]]

I would suggest to create a version-agnostic details page. With the rapid release cycle, it might be overlooked again by some locales, so it would be easier to simply have something to copy-paste.

Comment 8

[Rail Aliiev [:rail]]

I have pushed the fixed snippets to the betatest and releasetest channels.

I still need to investigate updates for the 3.6.x (<3.6.18). Looks like updates for those versions point to 5.0 instead of 3.6.18.

Comment 9

[Nick Thomas [:nthomas] (UTC+12)]

Rail, I read comment #6 to mean use '../5.0/details', but it looks like you've used '.../3.6/details/'. Was there some other reason to do that ?

pascalc has fixed up the details pages for locales now, eg
  https://www.mozilla.com/de/firefox/5.0/details/
  https://www.mozilla.com/tr/firefox/5.0/details/
  https://www.mozilla.com/fa/firefox/5.0/details/

Comment 10

[LegNeato]

(In reply to comment #7)
> I would suggest to create a version-agnostic details page. With the rapid
> release cycle, it might be overlooked again by some locales, so it would be
> easier to simply have something to copy-paste.

That is bug 668108

Comment 11

[Rail Aliiev [:rail]]

I have pushed the fixed snippets to the release and beta channels. The XML looks fine.