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)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: pascalc, Assigned: pascalc)
References
Details
Attachments
(1 file)
(deleted),
text/x-github-pull-request
|
Details |
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 | ||
Updated•11 years ago
|
Assignee: nobody → pascalc
Assignee | ||
Comment 1•11 years ago
|
||
Comment 2•11 years ago
|
||
Thanks for the PR! We can also denote GA once this change is merged and pushed.
Comment 3•11 years ago
|
||
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/
Assignee | ||
Updated•11 years ago
|
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Comment 4•11 years ago
|
||
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 → ---
Comment 5•11 years ago
|
||
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
Comment 6•11 years ago
|
||
Actually, the no-locale redirects seem to work fine. Redirects look good.
Comment 7•11 years ago
|
||
Closing this as bug 1009247 fixes this and I was commenting in the wrong bug. :)
Status: REOPENED → RESOLVED
Closed: 11 years ago → 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•