Closed
Bug 920268
Opened 11 years ago
Closed 11 years ago
Change /firefox/ redirect to point *only* to /firefox/new/
Categories
(www.mozilla.org :: Bedrock, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: cmore, Assigned: jgmize)
References
Details
(Whiteboard: [kb=1127721] )
Per bug 920212, we are going to change the /firefox/ redirect used by referral links and firefox.com to point to /firefox/new/ until the Firefox family website is live. /firefox/new/ works for all browsers and platforms and is our best converting page that is localized in the most languages. This should be a straight up redirect and also passing through query parameters from the referring URL and no more logic about the version of Firefox.
Reporter | ||
Comment 1•11 years ago
|
||
pmac/jgmize: this can be done sooner than the Firefox family page and doesn't have to wait for other things. This will make that redirect a lot more simple.
Updated•11 years ago
|
Whiteboard: [kb=1127721]
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → jmize
Assignee | ||
Comment 2•11 years ago
|
||
Submitted https://github.com/mozilla/bedrock/pull/1284
Comment 3•11 years ago
|
||
Commits pushed to master at https://github.com/mozilla/bedrock https://github.com/mozilla/bedrock/commit/21f97e7784d29f255c640cade8c31f37e554bf46 Fix Bug 920268 - Change /firefox/ redirect Point *only* to /firefox/new/, removing all version logic, while preserving querystrings, if present. Add name and query parameters to bedrock.redirects.util.redirect and change default behavior to pass query params to redirect location. Replace url('firefox') with url('firefox.new') in templates Remove unused bedrock/redirects/tests/urls.py https://github.com/mozilla/bedrock/commit/83d0b9e8bf4352f88a56e5abd62de93f8b92479c Merge pull request #1284 from jgmize/bug-920268 Fix Bug 920268 - Change /firefox/ redirect
Updated•11 years ago
|
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 4•11 years ago
|
||
This appears to be working here: https://www-dev.allizom.org/firefox/ > /firefox/new/ https://www-dev.allizom.org/en-US/firefox/ > /firefox/new/ https://www-dev.allizom.org/firefox/?test=1 > /firefox/new/?test=1 https://www-dev.allizom.org/firefox > /firefox/new/ Once this is pushed to production, I will annotate GA.
Comment 5•11 years ago
|
||
fixed on stage curl -ILA "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7; rv:17.0) Gecko/17.0 Firefox/17.0.8" http://www.allizom.org/firefox/ HTTP/1.1 301 Moved Permanently Content-Type: text/html Date: Thu, 10 Oct 2013 22:53:40 GMT Location: https://www.allizom.org/firefox/ Connection: Keep-Alive Content-Length: 0 HTTP/1.1 301 MOVED PERMANENTLY Server: Apache Vary: Accept-Language, Accept-Encoding X-Backend-Server: bedrock1.stage.webapp.scl3.mozilla.com Cache-Control: max-age=600 Content-Type: text/html; charset=utf-8 Date: Thu, 10 Oct 2013 22:43:52 GMT Location: https://www.allizom.org/en-US/firefox/ Expires: Thu, 10 Oct 2013 22:53:52 GMT Transfer-Encoding: chunked X-Frame-Options: DENY Connection: Keep-Alive X-Cache-Info: cached HTTP/1.1 301 MOVED PERMANENTLY 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: Thu, 10 Oct 2013 22:43:52 GMT Location: https://www.allizom.org/en-US/firefox/new/ Expires: Thu, 10 Oct 2013 22:53:52 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: Thu, 10 Oct 2013 22:43:52 GMT Expires: Thu, 10 Oct 2013 22:53:52 GMT Transfer-Encoding: chunked X-Frame-Options: DENY Connection: Keep-Alive X-Cache-Info: cached
Comment 6•11 years ago
|
||
fixed on prod curl -ILA "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7; rv:17.0) Gecko/17.0 Firefox/17.0.8" http://www.mozilla.org/firefox/ HTTP/1.1 301 MOVED PERMANENTLY Server: Apache Vary: Accept-Language, Accept-Encoding X-Backend-Server: bedrock1.webapp.phx1.mozilla.com Cache-Control: max-age=600 Content-Type: text/html; charset=utf-8 Date: Thu, 10 Oct 2013 23:10:41 GMT Location: http://www.mozilla.org/en-US/firefox/ Expires: Thu, 10 Oct 2013 23:20:41 GMT Transfer-Encoding: chunked Connection: Keep-Alive X-Frame-Options: DENY X-Cache-Info: caching HTTP/1.1 301 MOVED PERMANENTLY Server: Apache X-Backend-Server: bedrock2.webapp.phx1.mozilla.com Vary: Accept-Encoding Cache-Control: max-age=600 Content-Type: text/html; charset=utf-8 Date: Thu, 10 Oct 2013 23:10:41 GMT Location: http://www.mozilla.org/en-US/firefox/new/ Expires: Thu, 10 Oct 2013 23:20:41 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.webapp.phx1.mozilla.com Vary: Accept-Encoding Cache-Control: max-age=600 Content-Type: text/html; charset=utf-8 Date: Thu, 10 Oct 2013 23:10:41 GMT Expires: Thu, 10 Oct 2013 23:20:42 GMT Transfer-Encoding: chunked Connection: Keep-Alive X-Frame-Options: DENY X-Cache-Info: caching Cmore please mark as verified if you start seeing results in GA
Reporter | ||
Comment 7•11 years ago
|
||
Marked in GA. Thanks!
You need to log in
before you can comment on or make changes to this bug.
Description
•