Pascal Chevrel:pascalc (PTO until August 21) Assignee
	Description • 10 years ago

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

	Chris More [:cmore]
	Comment 2 • 10 years ago

Thanks for the PR! We can also denote GA once this change is merged and pushed.

	[github robot]
	Comment 3 • 10 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/

	Chris More [:cmore]
	Comment 4 • 10 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.

	Chris More [:cmore]
	Comment 5 • 10 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

	Chris More [:cmore]
	Comment 6 • 10 years ago

Actually, the no-locale redirects seem to work fine. Redirects look good.

	Chris More [:cmore]
	Comment 7 • 10 years ago

Closing this as bug 1009247 fixes this and I was commenting in the wrong bug. :)