1009247 - Redirect Firefox/Mobile Aurora/Beta PHP download pages to /firefox/channel/

Status: RESOLVED FIXED

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

Description

[Kohei Yoshino]

+++ This bug was initially created as a clone of Bug #1005237 +++

/firefox/beta/ -> /firefox/channel/#beta
/firefox/aurora/ -> /firefox/channel/#aurora

/mobile/beta/ -> /firefox/channel/#beta
/mobile/aurora/ -> /firefox/channel/#aurora

Redirect in all locales and remove the PHP pages.

Comment 2

[Kohei Yoshino]

Attachment: pull request

Comment 3

[Chris More [:cmore]]

* Remove:

"learn more about beta"

https://github.com/mozilla/bedrock/blob/master/bedrock/firefox/templates/firefox/channel.html#L41

"learn more about aurora"

https://github.com/mozilla/bedrock/blob/master/bedrock/firefox/templates/firefox/channel.html

* Any any links pointing to /firefox/beta/ and /firefox/aurora/ in:

https://github.com/mozilla/bedrock/blob/master/bedrock/firefox/templates/firefox/channel.html

* Remove aurora/beta redirects here:

https://github.com/mozilla/bedrock/blob/master/media/js/firefox/channel.js#L15

(We can keep the redirect for /firefox/ in the .js for the main channel as that redirects to /firefox/new/)

Comment 4

[Kohei Yoshino]

Updated my PR to remove/replace those links.

Comment 5

[[github robot]]

Commits pushed to master at https://github.com/mozilla/bedrock

https://github.com/mozilla/bedrock/commit/328ac861fe9ed6c1bb993a62cfd49818217255eb
Fix Bug 1009247 - Redirect Firefox/Mobile Aurora/Beta PHP download pages to /firefox/channel/

https://github.com/mozilla/bedrock/commit/d1ad870b0972c93300a6a822b84b39bdd403cf4c
Merge pull request #2000 from kyoshino/bug-1009247-channel-redirects

Fix Bug 1009247 - Redirect Firefox/Mobile Aurora/Beta PHP download pages to /firefox/channel/

Comment 6

[Chris More [:cmore]]

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

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 7

[Kohei Yoshino]

Attachment: patch for the PHP side redirects

Comment 8

[Kohei Yoshino]

Now these files can be removed from SVN:

en-US/firefox/aurora
en-US/firefox/aurora/index.html
en-US/firefox/beta
en-US/firefox/beta/index.html
en-US/img/firefox/beta
en-US/img/firefox/beta/3.1firstrun-title.png
en-US/mobile/aurora
en-US/mobile/aurora/details
en-US/mobile/aurora/details/index.html
en-US/mobile/aurora/firstrun
en-US/mobile/aurora/firstrun/index.html
en-US/mobile/aurora/index.html
en-US/mobile/beta
en-US/mobile/beta/index.html
img/covehead/firefox/aurora/background.jpg
img/covehead/firefox/aurora/blog-icons.png
img/covehead/firefox/aurora/divider.png
img/covehead/firefox/aurora/footer.png
img/covehead/firefox/aurora/mail.png
img/covehead/firefox/aurora/title.png
img/covehead/firefox/beta/bg-tablet.jpg
img/covehead/firefox/beta/blog-icons.png
img/covehead/firefox/beta/mail.png
img/covehead/firefox/beta/notes/bg.jpg
img/covehead/firefox/beta/social-icon.png
img/covehead/firefox/beta/title.png
img/covehead/mobile/aurora
img/covehead/mobile/aurora/bg-feature.jpg
img/covehead/mobile/aurora/bg.jpg
img/covehead/mobile/aurora/connected-icons.png
img/covehead/mobile/aurora/divider.jpg
img/covehead/mobile/aurora/install-steps.png
img/covehead/mobile/aurora/social-icon.png
img/firefox/12
img/firefox/12/firstrun
img/firefox/12/firstrun/aurora.png
img/firefox/12/firstrun/bg.jpg
img/firefox/beta
img/firefox/beta/4
img/firefox/beta/4/details
img/firefox/beta/4/details/background.jpg
img/firefox/beta/4/details/title.png
img/firefox/beta/4/disclaimer-panel.png
img/firefox/beta/4/download-button-release.png
img/firefox/beta/4/firstrun
img/firefox/beta/4/firstrun/l10npromo.jpg
img/firefox/beta/4/mobile-addons.png
img/firefox/beta/4/tech-table-borders.png
img/firefox/beta/4/tech-table-menu.png
img/firefox/beta/4/title.png
img/mobile/beta
img/mobile/beta/beta-install-inset.png
img/mobile/beta/beta-itstime-tshirt.png
img/tignish/firefox/background-all-beta-box-footer-ie.png
img/tignish/firefox/background-all-beta-box-footer.png
img/tignish/firefox/background-all-beta-box-ie.png
img/tignish/firefox/background-all-beta-box-inside-ie.png
img/tignish/firefox/background-all-beta-box-inside.png
img/tignish/firefox/background-all-beta-box.png
img/tignish/firefox/background-all-beta.jpg
img/tignish/whatsnew/background-whatsnew-beta.jpg
includes/feeds/AuroraFeed.php
includes/feeds/BetaFeed.php
includes/feeds/RapidReleaseFeed.php
includes/l10n/firefox
includes/l10n/firefox/aurora.inc.php
includes/l10n/firefox/beta.inc.php
includes/l10n/mobile
includes/l10n/mobile/beta.inc.php
js/mobile/beta
js/mobile/beta/gomobile.js
js/mobile/beta/main.js
style/covehead/aurora.css
style/covehead/beta.css
style/covehead/mobile-aurora.css
style/covehead/mobile-beta.css
style/firefox/aurora
style/firefox/aurora/firstrun-page.css
style/firefox/beta
style/firefox/beta/4
style/firefox/beta/4/main.css

Comment 9

[Kohei Yoshino]

pascalc: can you please verify the list in my comment 8 and remove those files?

Comment 10

[Pascal Chevrel:pascalc]

I can remove the files for l10n as I know them, but the files used by en-US are out of my scope.

Comment 11

[Steven Garrity [:sgarrity]]

A lot of these files are addressed in Bug 1027697.