876581 - Freshen up phishing-protection webpage

Status: RESOLVED FIXED

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

Description

[Raymond Lee [:raymondlee]]

Gavin suggests to freshen up the page see bug 846944 comment 12
http://www.mozilla.org/%LOCALE%/firefox/phishing-protection/

Comment 1

[:Gavin Sharp [email: [e-mail]]]

The page currently redirects to firefox/phishing-protection/firefox2/, which has the old design and refers to "Firefox 2".

Comment 2

[Andrei Hajdukewycz [:sancus]]

Should this page even be on www.mozilla.org, or would a SUMO article be more appropriate? I dunno, I'll leave it up to Jen to decide.

Comment 3

[Jennifer Bertsch [:jbertsch]]

I really like the idea of making this a sumo page.

Ibai, do we already have something related on SUMO?  Let's discuss when we meet up next week.

Comment 4

[Ibai Garcia [:ibai]]

Thumbs up to that. 

I don't think that we have anything specific about Phising. The closest articles is this:
https://support.mozilla.org/en-US/kb/how-stay-safe-web

And it covers a more general area. What are we looking to achieve with this page? Adding Michael Verdi who is our content manager.

Comment 5

[Jennifer Bertsch [:jbertsch]]

Hi Gavin and Raymond -  Do you know how people reach http://www.mozilla.org/en-US/firefox/phishing-protection/firefox2/ from within the product?

Ibai and Michael - My goal would be to have an easy-to-update, contextually appropriate page about phishing for Fx browser users to reach in the scenario that Gavin or Raymond will describe to us :)

Thanks,
Jen

Comment 6

[Raymond Lee [:raymondlee]]

(In reply to jbertsch from comment #5)
> Hi Gavin and Raymond -  Do you know how people reach
> http://www.mozilla.org/en-US/firefox/phishing-protection/firefox2/ from
> within the product?

Visit a phishing site (e.g. http://www.mozilla.org/firefox/its-a-trap.html) and click on "Why was this page blocked?" button on the warning page.

BTW, if you visit http://www.mozilla.org/en-US/firefox/phishing-protection/ in other browser e.g. google chrome, it doesn't redirect to http://www.mozilla.org/en-US/firefox/phishing-protection/firefox2/ page and show a different page.

Comment 7

[Verdi [:verdi] Best to slack me]

(In reply to jbertsch from comment #3)
> I really like the idea of making this a sumo page.

We can certainly do that. We haven't had an article on sumo because this page covered it. I can reuse much of this copy. 


(In reply to Ibai Garcia [:ibai] from comment #4)
>The closest articles is this:
> https://support.mozilla.org/en-US/kb/how-stay-safe-web
> 
> And it covers a more general area.

Right. We can link this "new" phishing article from that more general article and from http://support.mozilla.org/kb/how-does-firefox-android-provide-secure-mobile-bro

Comment 8

[Jennifer Bertsch [:jbertsch]]

Fabulous.

Please let us know when the new content is ready, and we'll update the link.

We can also open bugs for localization of http://www.mozilla.org/firefox/its-a-trap.html when localized versions of the sumo page are ready.

Comment 9

[Kohei Yoshino]

> userAgentRedirect('Firefox/2', '/en-US/firefox/phishing-protection/firefox2/');

Ouch.

Comment 10

[:Gavin Sharp [email: [e-mail]]]

(In reply to Kohei Yoshino from comment #9)
> > userAgentRedirect('Firefox/2', '/en-US/firefox/phishing-protection/firefox2/');
> 
> Ouch.

Ouch indeed. Can we just fix this bad redirect quickly, separate from the larger issue of what to do with this page longer-term?

Comment 11

[Kohei Yoshino]

Attachment: Remove redirect and content for Firefox 2

Comment 12

[:Gavin Sharp [email: [e-mail]]]

Comment on attachment 758963 [details] [diff] [review]
Remove redirect and content for Firefox 2

Can we just get this landed?

Comment 13

[Andrei Hajdukewycz [:sancus]]

Comment on attachment 758963 [details] [diff] [review]
Remove redirect and content for Firefox 2

I don't see any reason we can't land this in the meantime while we wait for the SUMO content and a rewrite patch.

r116702. Please confirm that it's acceptable on dev(~20 mins after this comment) - https://www-dev.allizom.org/en-US/firefox/phishing-protection/ and I'll push live tomorrow.

Comment 14

[Kohei Yoshino]

(In reply to Jennifer Bertsch [:jbertsch] from comment #8)
> We can also open bugs for localization of
> http://www.mozilla.org/firefox/its-a-trap.html when localized versions of
> the sumo page are ready.

This page has been moved to http://www.itisatrap.org/ (Bug 662213, Bug 710597)

Comment 15

[Andrei Hajdukewycz [:sancus]]

(In reply to Kohei Yoshino from comment #14)
> This page has been moved to http://www.itisatrap.org/ (Bug 662213, Bug
> 710597)

That also sort of stalled in Bug 662046.

Comment 16

[Chris More [:cmore]]

(In reply to Andrei Hajdukewycz [:sancus] from comment #2)
> Should this page even be on www.mozilla.org, or would a SUMO article be more
> appropriate? I dunno, I'll leave it up to Jen to decide.

I want these off of mozilla.org. I comment bug 662046 as I keep going round and round on what is actually blocking what.

Comment 18

[Chris More [:cmore]]

Would SUMO be able to create a page with the same content at: http://www.mozilla.org/en-US/firefox/phishing-protection ? I don't think it is worth porting this to bedrock when it is just about a feature in Firefox and it would be easier updated in the future.

It is linked from the Firefox products here:


b2g/app/b2g.js

pref("browser.safebrowsing.warning.infoURL", "http://www.mozilla.com/%LOCALE%/%APP%/phishing-protection/");

browser/app/profile/firefox.js

pref("browser.safebrowsing.warning.infoURL", "https://www.mozilla.org/%LOCALE%/firefox/phishing-protection/");

browser/metro/profile/metro.js

pref("browser.safebrowsing.warning.infoURL", "https://www.mozilla.org/%LOCALE%/firefox/phishing-protection/");

mobile/android/app/mobile.js

pref("browser.safebrowsing.warning.infoURL", "http://www.mozilla.com/%LOCALE%/firefox/phishing-protection/");

Comment 19

[Chris More [:cmore]]

Jen: Can you make a decision and work with SUMO this content on their website and after that we can update the products and redirect this page?

Comment 20

[Jennifer Bertsch [:jbertsch]]

Yes-

SUMO has created the content.

We can point to:
Phishing: https://support.mozilla.org/en-US/kb/how-does-phishing-and-malware-protection-work#w_what-is-a-web-forgery-what-is-phishing
Malware:  https://support.mozilla.org/en-US/kb/how-does-phishing-and-malware-protection-work#w_what-is-an-attack-site-what-is-malware

Thanks, cmore!

Comment 21

[Chris More [:cmore]]

The issue is that we need a single URL that has links to both of this content? The product points to a single mozilla.org page that has info on both phishing and malware. Can SUMO create a single meta page?

Comment 22

[Jennifer Bertsch [:jbertsch]]

Cmore-

Here you go:  https://support.mozilla.org/en-US/kb/how-does-phishing-and-malware-protection-work

Comment 23

[Chris More [:cmore]]

Does everyone support redirecting:

http://www.mozilla.org/en-US/firefox/phishing-protection/

to:

https://support.mozilla.org/en-US/kb/how-does-phishing-and-malware-protection-work

If so, I will submit the IT bug to have this done.

The next step would be to submit a bug to the Firefox team to update references here:

http://dxr.mozilla.org/mozilla-central/search?tree=mozilla-central&q=phishing-protection&redirect=true

Comment 24

[Jennifer Bertsch [:jbertsch]]

I totally support it.  Let's do it!

Comment 25

[Steven Garrity [:sgarrity]]

Based on comment #23, I'll take this, set up the redirect, and then remove the page from SVN.

Comment 26

[Steven Garrity [:sgarrity]]

Redirect PR: https://github.com/mozilla/bedrock/pull/1381

Also note that there is an Italian translation that will get redirected here as well (https://www.mozilla.org/it/firefox/phishing-protection/).

Comment 27

[Verdi [:verdi] Best to slack me]

Please make sure when you link to sumo you do it without the locale string. If you use https://support.mozilla.org/kb/how-does-phishing-and-malware-protection-work SUMO will send users to the localized version of the article.

Comment 28

[Steven Garrity [:sgarrity]]

(In reply to Verdi [:verdi] from comment #27)
> Please make sure when you link to sumo you do it without the locale string.

Yup, I've left out the locale string in the pull-request. Thanks!

Comment 29

[[github robot]]

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

https://github.com/mozilla/bedrock/commit/8b4c22455645008e412952100481e3321ada0354
Redirect phishing-protection page to sumo
Bug 876581

Comment 30

[raymond [:retornam] (inactive)]

fixed on stage https://support.mozilla.org/en-US/kb/how-does-phishing-and-malware-protection-work

Comment 31

[Steven Garrity [:sgarrity]]

Attachment: bug-876581-remove-phishing.diff

Now that the phishing-protection page is redirected to SUMO, this patch removes the related files from svn.

Comment 32

[Francesco Lodolo [:flod]]

Interesting, no idea why we're the only locale with that page (and I'm even responsible for localizing it 4 years ago...). For me it's safe to remove these files.

Comment 33

[Steven Garrity [:sgarrity]]

In trunk in r122524. Merged into tags/stage in r122528, and tags/production in r122529.