Closed Bug 1710891 Opened 4 years ago Closed 4 years ago

Link to download full build from About dialog is broken

Categories

(Firefox :: General, defect)

Firefox 90
Desktop
Windows 10
defect

Tracking

()

RESOLVED FIXED
Tracking Status
firefox90 --- affected

People

(Reporter: ian.henderson, Assigned: nrishel)

References

Details

(Whiteboard: [fidedi-ope])

Attachments

(1 file)

Attached image Automatic update

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

When I try to update from the autoupdate the URL https://www.mozilla.org/en-GB/sco/firefox/channel/desktop/#nightly does not appear to be correct.

For reference:

  • This happens when the update installation fails, and the about dialog displays a link to download a new build.
  • This is a sco build, which doesn't have localized pages on mozilla.org, and has an accept-languages header set as sco,en-GB,en.

So, I would expect Firefox to ask for https://www.mozilla.org/firefox/channel/desktop/#nightly. Since sco is not available, it should fall back to en-GB. If I manually set intl.accept_languages to sco,en-GB,en and open the locale agnostic URL in the browser, it works as expected.

Component: Application Update → General
Product: Toolkit → Firefox
Summary: Update of nightly URL appears wrong → Link to download full build from About dialog is broken
No longer depends on: 1710890

(In reply to Francesco Lodolo [:flod] from comment #1)

So, I would expect Firefox to ask for https://www.mozilla.org/firefox/channel/desktop/#nightly. Since sco is not available, it should fall back to en-GB. If I manually set intl.accept_languages to sco,en-GB,en and open the locale agnostic URL in the browser, it works as expected.

Are you sure that that is the behavior that we want here? The page is arguably localized correctly, but the download link will be for the en-GB version of Firefox, not the sco version of Firefox.

Severity: -- → S3
Flags: needinfo?(francesco.lodolo)

That's not the behavior we want, that's the best that we can get right now.

There have discussions around it for MR1, but considered out of scope. IMO, the link should just download the same build as the one used by the user. In case, Romain would be the person to talk to about scoping and fixing this properly.

Flags: needinfo?(francesco.lodolo)
Whiteboard: [fidedi-ope]

https://bugzilla.mozilla.org/show_bug.cgi?id=1730110#c2 suggests a solution to a tangential problem that would allow for the language granularity needed here, at the cost of being directed to a page which isn't reengaging. Maybe that's less necessary for nightly, where there's more risk of mismatch between mozilla.org and Firefox locales?

Flags: needinfo?(pmac)

(In reply to Nick Rishel from comment #4)

Maybe that's less necessary for nightly, where there's more risk of mismatch between mozilla.org and Firefox locales?

I don't think the risk is that smaller outside of Nightly: population is larger, and we have plenty of locales falling back with localization of mozilla.org (or not being there at all).

Thanks for the ping. I think the original bug should be fixed now since the website does now support the sco locale, but this page isn't translated into sco so they would still just get the en-GB version of the page, and thus the en-GB version of the browser. We could send them to https://www.mozilla.org/en-US/firefox/all/#product-desktop-nightly instead so they can select their language, but that page can be confusing to some because of all of the choices of browser and platform.

Flags: needinfo?(pmac)
Assignee: nobody → nrishel
Blocks: 1751271
No longer blocks: 1751271

Closing as the original bug was fixed when the sco locale was added to bedrock. Follow up bug report for locale download mismatch opened here - https://bugzilla.mozilla.org/show_bug.cgi?id=1751271.

Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: