845983 - Redirect /metrofirefox/foo to /firefox/foo on www.mozilla.org

Status: VERIFIED FIXED

(www.mozilla.org :: General, defect, P2)

Description

[Tim Abraldes [:TimAbraldes] [:tabraldes]]

When users click the "Why was this page blocked" button on about:blocked, we send them to a URL specified by the pref "browser.safebrowsing.warning.infoURL", which defaults to "http://www.mozilla.org/%LOCALE%/%APP%/phishing-protection"

On my machine, this points to "http://www.mozilla.org/en-US/firefox/phishing-protection" for desktop Firefox and to "http://www.mozilla.org/en-US/metrofirefox/phishing-protection" for Metro/Immersive Firefox.

The latter URL is currently a 404, but we'll need it to be functional when we release Metro/Immersive Firefox.  I believe it is sufficient to have the same content as (or maybe have a redirect to) the desktop Firefox page.  CCing Asa to make calls about the content of this page.

We probably have a similar situation with other URLs.

Comment 1

[Jennifer Bertsch [:jbertsch]]

Cmore: Could you please grab the 404 stats for this page?

Tim: What is the timeline to release metro/immersive Firefox?

Thx,
Jen

Comment 2

[Chris More [:cmore]]

I checked and over the past 30 days we have not received this 404 error except for the ones generated today.

Can we put in a 301 redirect from /[locale]/metrofirefox/* to /[locale]/firefox/* ?

I don't expect to have a /metrofirefox/ directory at any point and may be easier to ship all the traffic over to the main /firefox directory. Currently, /firefox is a redirect to /firefox/fx/ or /firefox/new depending on the user agent, but in the next quarter /firefox/ will be the family page for all Firefox products on all platforms.

Comment 3

[Asa Dotzler [:asa]]

We can point at the same content. How we do that isn't important to me.

Comment 4

[Tim Abraldes [:TimAbraldes] [:tabraldes]]

(In reply to Chris More [:cmore] from comment #2)
> I checked and over the past 30 days we have not received this 404 error
> except for the ones generated today.

Part of the reason we haven't seen any of these 404s is bug 840776, which made it so that the page would never be requested.

Also, I imagine it is infrequent for users to reach the about:blocked page, and since we have so few users on Metro/Immersive Firefox it seems unlikely that anyone would have hit the 404 by now.

> Can we put in a 301 redirect from /[locale]/metrofirefox/* to
> /[locale]/firefox/* ?

I think a 301 probably makes sense.


(In reply to jbertsch from comment #1)
> Tim: What is the timeline to release metro/immersive Firefox?

I don't think a definite timeline has been established but metro/immersive Firefox is already on m-c which means Nightly users are currently using it.

Comment 5

[Asa Dotzler [:asa]]

(In reply to Tim Abraldes (:tabraldes) from comment #4)
> (In reply to jbertsch from comment #1)
> > Tim: What is the timeline to release metro/immersive Firefox?
> 
> I don't think a definite timeline has been established but metro/immersive
> Firefox is already on m-c which means Nightly users are currently using it.

cmore, it would be great if we could get the redirect in place within the next ten days. Is that feasible?

Comment 6

[Chris More [:cmore]]

(In reply to Tim Abraldes (:tabraldes) from comment #4)
> (In reply to Chris More [:cmore] from comment #2)
> > I checked and over the past 30 days we have not received this 404 error
> > except for the ones generated today.
> 
> Part of the reason we haven't seen any of these 404s is bug 840776, which
> made it so that the page would never be requested.
> 
> Also, I imagine it is infrequent for users to reach the about:blocked page,
> and since we have so few users on Metro/Immersive Firefox it seems unlikely
> that anyone would have hit the 404 by now.
> 
> > Can we put in a 301 redirect from /[locale]/metrofirefox/* to
> > /[locale]/firefox/* ?
> 
> I think a 301 probably makes sense.
> 
> 
> (In reply to jbertsch from comment #1)
> > Tim: What is the timeline to release metro/immersive Firefox?
> 
> I don't think a definite timeline has been established but metro/immersive
> Firefox is already on m-c which means Nightly users are currently using it.

I figured that is why we aren't getting many 404's now. :)

We can get this redirect on an upcoming release cycle.

Probably something like this:

RewriteRule ^(.+)/(metrofirefox)(.+)/ $1/firefox$3 [R=permanent]

That should redirect:

/[locale]/metrofirefox* to /[locale]/firefox*

Base case: https://www.mozilla.org/metrofirefox/ should redirect to /firefox/

Also anything after /metrofirefox/ should redirect to /firefox/whatever, as in https://www.mozilla.org/metrofirefox/foo/ should redirect to /firefox/foo/.

Comment 7

[Chris More [:cmore]]

Just to confirm, can we redirect /metrofirefox/* to /firefox/*? Since /metrofirefox/ doesn't even exist, I don't see a reason to redirect only the phishing web page.

Comment 8

[Tim Abraldes [:TimAbraldes] [:tabraldes]]

(In reply to Chris More [:cmore] from comment #7)
> Just to confirm, can we redirect /metrofirefox/* to /firefox/*? Since
> /metrofirefox/ doesn't even exist, I don't see a reason to redirect only the
> phishing web page.

That sounds like it should be fine, but I don't really know anything about serving pages on mozilla.org and I'm not sure if you were directing that question at me :)

Comment 9

[Chris More [:cmore]]

(In reply to Tim Abraldes (:tabraldes) from comment #8)
> (In reply to Chris More [:cmore] from comment #7)
> > Just to confirm, can we redirect /metrofirefox/* to /firefox/*? Since
> > /metrofirefox/ doesn't even exist, I don't see a reason to redirect only the
> > phishing web page.
> 
> That sounds like it should be fine, but I don't really know anything about
> serving pages on mozilla.org and I'm not sure if you were directing that
> question at me :)

It was to you and Asa if you are fine with making a generic redirect for metrofirefox instead of making it only work for the phishing page.

Comment 10

[Asa Dotzler [:asa]]

(In reply to Chris More [:cmore] from comment #9)
> (In reply to Tim Abraldes (:tabraldes) from comment #8)
> > (In reply to Chris More [:cmore] from comment #7)
> > > Just to confirm, can we redirect /metrofirefox/* to /firefox/*? Since
> > > /metrofirefox/ doesn't even exist, I don't see a reason to redirect only the
> > > phishing web page.
> > 
> > That sounds like it should be fine, but I don't really know anything about
> > serving pages on mozilla.org and I'm not sure if you were directing that
> > question at me :)
> 
> It was to you and Asa if you are fine with making a generic redirect for
> metrofirefox instead of making it only work for the phishing page.

I'm not familiar with what all lives at /firefox/* and is likely to be triggered by our Metro browser. That being said, tt probably wouldn't hurt to do the full directory level redirect.

Comment 11

[Jennifer Bertsch [:jbertsch]]

Hi Chris-  Do we have a clear way forward?  Does this go to Pmac next?

Thx,
Jen

Comment 12

[Chris More [:cmore]]

(In reply to jbertsch from comment #11)
> Hi Chris-  Do we have a clear way forward?  Does this go to Pmac next?
> 
> Thx,
> Jen

Yes, comment 6 should be fine. Just redirect all of /metrofirefox/ to /firefox/ and any URL after /metrofirefox/ should be carried through to /firefox/.

Pmac could work on it, but it is not as mission critical as other fires we are putting out now.

Comment 13

[[github robot]]

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

https://github.com/mozilla/bedrock/commit/5ec19b72ec45a6541ec9b4bc185895468900780d
Bug 845983: Redirect metrofirefox/* to firefox/*

Comment 14

[raymond [:retornam] (inactive)]

fixed on stage  https://www.allizom.org/metrofirefox --> https://www.allizom.org/en-US/firefox/new/

Comment 15

[raymond [:retornam] (inactive)]

verified fixed http://www.mozilla.org/metrofirefox -->  http:/www.mozilla.org/firefox/new/