Change /firefox/ redirect to point *only* to /firefox/new/

VERIFIED FIXED

Status

www.mozilla.org
Bedrock
VERIFIED FIXED
4 years ago
4 years ago

People

(Reporter: cmore, Assigned: jgmize)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [kb=1127721] )

(Reporter)

Description

4 years ago
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

4 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.
Whiteboard: [kb=1127721]
(Assignee)

Updated

4 years ago
Assignee: nobody → jmize
(Assignee)

Comment 2

4 years ago
Submitted https://github.com/mozilla/bedrock/pull/1284

Comment 3

4 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

4 years ago
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
(Reporter)

Comment 4

4 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.
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
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

4 years ago
Marked in GA. Thanks!
verified fixed
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.