Closed Bug 894074 Opened 11 years ago Closed 11 years ago

Redirects broken on stage for locales other than en-US

Categories

(www.mozilla.org :: Bedrock, defect)

Product:
www.mozilla.org
Component:
Bedrock
Platform:
x86
macOS
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: retornam, Unassigned)

References

(
URL
)

Details

(Whiteboard: [fromAutomation]) 

STR:

When a user is using an old version of Firefox for example Firefox 16 and a locale other than en-US (eg fr, de), the user is redirected to https://www.allizom.org/en-US/firefox/update/  instead of the locale specific page at https://www.allizom.org/de/firefox/update/ or https://www.allizom.org/fr/firefox/update/

curl -H  "Accept-Language: de" -ILA "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7; rv:16.0) Gecko/16.0 Firefox/16.0" www.allizom.org/de/firefox/
HTTP/1.1 301 Moved Permanently
Content-Type: text/html
Date: Mon, 15 Jul 2013 21:53:01 GMT
Location: https://www.allizom.org/de/firefox/
Connection: Keep-Alive
Content-Length: 0

HTTP/1.1 301 MOVED PERMANENTLY
Server: Apache
Vary: User-Agent, Accept-Encoding
X-Backend-Server: bedrock1.stage.webapp.scl3.mozilla.com
Cache-Control: max-age=600
Content-Type: text/html; charset=utf-8
Date: Mon, 15 Jul 2013 21:51:35 GMT
Location: https://www.allizom.org/de/firefox/update/
Expires: Mon, 15 Jul 2013 22:01:35 GMT
Transfer-Encoding: chunked
x-frame-options: DENY
Connection: Keep-Alive
X-Cache-Info: cached

HTTP/1.1 302 FOUND
Server: Apache
X-Backend-Server: bedrock1.stage.webapp.scl3.mozilla.com
Vary: Accept-Encoding
Cache-Control: max-age=600
Content-Type: text/html; charset=utf-8
Date: Mon, 15 Jul 2013 21:51:35 GMT
Location: https://www.allizom.org/en-US/firefox/update/
Expires: Mon, 15 Jul 2013 22:01:35 GMT
Transfer-Encoding: chunked
x-frame-options: DENY
Connection: Keep-Alive
X-Cache-Info: cached

HTTP/1.1 200 OK
Server: Apache
X-Backend-Server: bedrock1.stage.webapp.scl3.mozilla.com
Vary: Accept-Encoding
Cache-Control: max-age=600
Content-Type: text/html; charset=utf-8
Date: Mon, 15 Jul 2013 21:51:35 GMT
Expires: Mon, 15 Jul 2013 22:01:35 GMT
Transfer-Encoding: chunked
x-frame-options: DENY
Connection: Keep-Alive
X-Cache-Info: cached




Everything is fine when the user is using a current release version of Firefox.


curl -H  "Accept-Language: de" -ILA "Macintosh; Intel Mac OS X 10.7; rv:22.0) Gecko/22.0 Firefox/22.0" www.allizom.org/de/firefox/
HTTP/1.1 301 Moved Permanently
Content-Type: text/html
Date: Mon, 15 Jul 2013 21:56:00 GMT
Location: https://www.allizom.org/de/firefox/
Connection: Keep-Alive
Content-Length: 0

HTTP/1.1 302 FOUND
Server: Apache
Vary: User-Agent, Accept-Encoding
X-Backend-Server: bedrock1.stage.webapp.phx1.mozilla.com
Cache-Control: max-age=600
Content-Type: text/html; charset=utf-8
Date: Mon, 15 Jul 2013 21:56:00 GMT
Location: https://www.allizom.org/de/firefox/fx/
Expires: Mon, 15 Jul 2013 22:06:00 GMT
Transfer-Encoding: chunked
Connection: Keep-Alive
x-frame-options: DENY
X-Cache-Info: caching

HTTP/1.1 200 OK
Server: Apache
X-Backend-Server: bedrock1.stage.webapp.phx1.mozilla.com
Vary: Accept-Encoding
Cache-Control: max-age=900
Content-Type: text/html; charset=UTF-8
Date: Mon, 15 Jul 2013 21:56:00 GMT
Expires: Mon, 15 Jul 2013 22:11:00 GMT
Transfer-Encoding: chunked
X-Robots-Tag: noodp
Connection: Keep-Alive
X-Cache-Info: caching
Blocks: 893006
Blocks: 892716
No longer blocks: 893006
Commits pushed to master at https://github.com/mozilla/bedrock

https://github.com/mozilla/bedrock/commit/edda4dec638d1fda802645e3311ec4ac0dd03e8b
Bug 894074: /firefox should redirect to fx/new/ for old fx.

Also fix bug 869937: Similar redirect semantics for
whatsnew and firstrun pages.

https://github.com/mozilla/bedrock/commit/6b257a82b62ee931749ed10857562600476aa658
Merge pull request #1070 from pmclanahan/change-firefox-update-redirects-new-894074

Bug 894074: /firefox should redirect to fx/new/ for old fx.
fixed on stage 


curl -H  "Accept-Language: de" -ILA "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7; rv:16.0) Gecko/16.0 Firefox/16.0" www.allizom.org/de/firefox/
HTTP/1.1 301 Moved Permanently
Content-Type: text/html
Date: Tue, 16 Jul 2013 16:56:28 GMT
Location: https://www.allizom.org/de/firefox/
Connection: Keep-Alive
Content-Length: 0

HTTP/1.1 301 MOVED PERMANENTLY
Server: Apache
Vary: User-Agent, Accept-Encoding
X-Backend-Server: bedrock1.stage.webapp.scl3.mozilla.com
Cache-Control: max-age=600
Content-Type: text/html; charset=utf-8
Date: Tue, 16 Jul 2013 16:56:09 GMT
Location: https://www.allizom.org/de/firefox/new/
Expires: Tue, 16 Jul 2013 17:06:10 GMT
Transfer-Encoding: chunked
x-frame-options: DENY
Connection: Keep-Alive
X-Cache-Info: cached

HTTP/1.1 200 OK
Server: Apache
X-Backend-Server: bedrock1.stage.webapp.scl3.mozilla.com
Vary: Accept-Encoding
Cache-Control: max-age=600
Content-Type: text/html; charset=utf-8
Date: Tue, 16 Jul 2013 16:56:10 GMT
Expires: Tue, 16 Jul 2013 17:06:12 GMT
Transfer-Encoding: chunked
x-frame-options: DENY
Connection: Keep-Alive
X-Cache-Info: cached




curl -H  "Accept-Language: de" -ILA "Macintosh; Intel Mac OS X 10.7; rv:22.0) Gecko/22.0 Firefox/22.0" www.allizom.org/de/firefox/
HTTP/1.1 301 Moved Permanently
Content-Type: text/html
Date: Tue, 16 Jul 2013 16:56:44 GMT
Location: https://www.allizom.org/de/firefox/
Connection: Keep-Alive
Content-Length: 0

HTTP/1.1 302 FOUND
Server: Apache
Vary: User-Agent, Accept-Encoding
X-Backend-Server: bedrock1.stage.webapp.scl3.mozilla.com
Cache-Control: max-age=600
Content-Type: text/html; charset=utf-8
Date: Tue, 16 Jul 2013 16:56:44 GMT
Location: https://www.allizom.org/de/firefox/fx/
Expires: Tue, 16 Jul 2013 17:06:45 GMT
Transfer-Encoding: chunked
Connection: Keep-Alive
x-frame-options: DENY
X-Cache-Info: caching

HTTP/1.1 200 OK
Server: Apache
X-Backend-Server: bedrock1.stage.webapp.scl3.mozilla.com
Vary: Accept-Encoding
Cache-Control: max-age=900
Content-Type: text/html; charset=UTF-8
Date: Tue, 16 Jul 2013 16:56:45 GMT
Expires: Tue, 16 Jul 2013 17:11:45 GMT
Transfer-Encoding: chunked
X-Robots-Tag: noodp
Connection: Keep-Alive
X-Cache-Info: caching
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Redirects work on prod now. I'm leaving this until I check in a test for it
You need to log in before you can comment on or make changes to this bug.