Closed Bug 1006616 Opened 11 years ago Closed 11 years ago

Redirect {locale}/download/ to {locale}/firefox/new

Categories

(www.mozilla.org :: Pages & Content, defect)

Production
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: pascalc, Assigned: pascalc)

References

Details

Attachments

(1 file)

We no longer use the interstitial page we had for downloads initiate from our download pages such as: https://www.mozilla.org/de/download/?product=firefox-29.0&os=win&lang=de We should redirect all traffic to these pages to firefox/new/, according to cmore this is mostly search engines. That would allow us to delete this old content on the PHP side
Assignee: nobody → pascalc
Thanks for the PR! We can also denote GA once this change is merged and pushed.
Commits pushed to master at https://github.com/mozilla/bedrock https://github.com/mozilla/bedrock/commit/c9b08b6b79dc9e97a8df493a4328cb3802093b33 Bug 1006616: redirect {locale}/download/* to {locale}/firefox/new/ https://github.com/mozilla/bedrock/commit/c0e8b2cdd958df550c76b6329c2495e15071e42b Merge pull request #1974 from pascalchevrel/Bug-1006616-redirect-localized-interstitial-download-page Bug 1006616: redirect {locale}/download/* to {locale}/firefox/new/
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
This broke non-en-US beta download pages. Example: http://www.mozilla.org/de/firefox/beta/ I checked stats on people visiting /firefox/new/ with ?product in the URL that is caused by the above pull request and it is affecting about 16,000 requests a day. Let's comment out or remove the redirect in comment 3 until we have a better solution.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Depends on: 1009247
Redirects look good: curl -IL https://www-dev.allizom.org/en-US/firefox/aurora/ HTTP/1.1 301 Moved Permanently Server: Apache X-Backend-Server: bedrock1.dev.webapp.phx1.mozilla.com Vary: Accept-Encoding Content-Type: text/html; charset=iso-8859-1 Date: Tue, 13 May 2014 00:03:06 GMT Location: https://www-dev.allizom.org/en-US/firefox/channel/#aurora Transfer-Encoding: chunked Connection: Keep-Alive HTTP/1.1 200 OK Server: Apache X-Backend-Server: bedrock1.dev.webapp.phx1.mozilla.com Vary: Accept-Encoding Cache-Control: max-age=600 Content-Type: text/html; charset=utf-8 Date: Tue, 13 May 2014 00:03:06 GMT Expires: Tue, 13 May 2014 00:13:07 GMT Transfer-Encoding: chunked Connection: Keep-Alive X-Robots-Tag: noodp X-Frame-Options: DENY curl -IL https://www-dev.allizom.org/en-US/firefox/beta/ HTTP/1.1 301 Moved Permanently Server: Apache X-Backend-Server: bedrock1.dev.webapp.phx1.mozilla.com Vary: Accept-Encoding Content-Type: text/html; charset=iso-8859-1 Date: Tue, 13 May 2014 00:04:36 GMT Location: https://www-dev.allizom.org/en-US/firefox/channel/#beta Transfer-Encoding: chunked Connection: Keep-Alive HTTP/1.1 200 OK Server: Apache X-Backend-Server: bedrock1.dev.webapp.phx1.mozilla.com Vary: Accept-Encoding Cache-Control: max-age=600 Content-Type: text/html; charset=utf-8 Date: Tue, 13 May 2014 00:04:36 GMT Expires: Tue, 13 May 2014 00:14:36 GMT Transfer-Encoding: chunked Connection: Keep-Alive X-Robots-Tag: noodp X-Frame-Options: DENY curl -IL https://www-dev.allizom.org/en-US/firefox/beta/?utm_source=test HTTP/1.1 301 Moved Permanently Server: Apache X-Backend-Server: bedrock1.dev.webapp.phx1.mozilla.com Vary: Accept-Encoding Content-Type: text/html; charset=iso-8859-1 Date: Tue, 13 May 2014 00:05:05 GMT Location: https://www-dev.allizom.org/en-US/firefox/channel/#beta?utm_source=test Transfer-Encoding: chunked Connection: Keep-Alive HTTP/1.1 200 OK Server: Apache X-Backend-Server: bedrock1.dev.webapp.phx1.mozilla.com Vary: Accept-Encoding Cache-Control: max-age=600 Content-Type: text/html; charset=utf-8 Date: Tue, 13 May 2014 00:05:05 GMT Expires: Tue, 13 May 2014 00:15:05 GMT Transfer-Encoding: chunked Connection: Keep-Alive X-Robots-Tag: noodp X-Frame-Options: DENY The only redirects that don't pass along #beta or #aurora is when you don't include the locale, which I think is the locale redirect and not this redirect: curl -IL https://www-dev.allizom.org/firefox/aurora/ HTTP/1.1 301 Moved Permanently Server: Apache X-Backend-Server: bedrock1.dev.webapp.phx1.mozilla.com Vary: Accept-Encoding Content-Type: text/html; charset=iso-8859-1 Date: Tue, 13 May 2014 00:06:27 GMT Location: https://www-dev.allizom.org/firefox/channel/#aurora Transfer-Encoding: chunked Connection: Keep-Alive HTTP/1.1 301 Moved Permanently Server: Apache X-Backend-Server: bedrock1.dev.webapp.phx1.mozilla.com Vary: User-Agent,Accept-Language Vary: Accept-Encoding Cache-Control: no-store, no-cache, must-revalidate, post-check=0, pre-check=0, private Content-Type: text/html; charset=UTF-8 Date: Tue, 13 May 2014 00:06:27 GMT Location: https://www-dev.allizom.org/en-US/firefox/channel/ Pragma: no-cache Transfer-Encoding: chunked Connection: Keep-Alive X-Robots-Tag: noodp
Actually, the no-locale redirects seem to work fine. Redirects look good.
Closing this as bug 1009247 fixes this and I was commenting in the wrong bug. :)
Status: REOPENED → RESOLVED
Closed: 11 years ago11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: