STR: 1) Visit http://www.mozilla.org/en-us/firefox/new/ 2) It redirects to http://www.mozilla.org/en-US/en-us/firefox/new/ If my memory serves me right lowercase locales redirected t the correct pages.
no, it never did, I don't mind if it does although I am not sure it is very useful.
and resetting to minor since we never had this feature.
Severity: critical → minor
Created attachment 8369145 [details] [review] Pull Request on GitHub
Assignee: nobody → kohei.yoshino
Status: NEW → ASSIGNED
Cmore or Gareth, We need more information on the number of users who hit 404 pages when they use lowercase letters for locales. It looks like this bug happens on a few locales only
I'll take a look at this...
(In reply to Gareth Cull [:garethc] from comment #5) > I'll take a look at this... Thanks Gareth.
I believe I've figured this one out. The URLs that are having the issue are in global.conf and are being passed through to bedrock but only for en-US. e.g.... https://github.com/mozilla/bedrock/blob/master/etc/httpd/global.conf#L182 This rule is case sensitive, so it's not forwarding requests to /en-us/firefox/new/ to bedrock. We can fix this either by making these rules case insensitive, or going with :kohei's patch. My vote is for the latter, so I'll start testing it to make sure it really fixes the issue. Thanks again :kohei
Thanks :pmac for the investigation!
Commits pushed to master at https://github.com/mozilla/bedrock https://github.com/mozilla/bedrock/commit/fd16dc8c0e118978b61659f93330c18310c87ce5 Fix Bug 795970 - using lowercase locales in url leads to 404 https://github.com/mozilla/bedrock/commit/202cbbfd2e897017c644517fcd507f7133b5585a Merge pull request #1652 from kyoshino/bug-795970-lowercase-locales Fix Bug 795970 - using lowercase locales in url leads to 404
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
fixed lowercase en-us for example goes to https://www.allizom.org/en-US/firefox/new/
You need to log in before you can comment on or make changes to this bug.