Update urlclassifier DB for Mozilla com->org move

VERIFIED FIXED in Firefox 10

Status

()

Toolkit
Safe Browsing
--
minor
VERIFIED FIXED
6 years ago
3 years ago

People

(Reporter: gcp, Assigned: gcp)

Tracking

Trunk
Firefox 11
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox9+, firefox10+ fixed)

Details

(Whiteboard: [mozmill-test-blocked])

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
After the move of all mozilla content from mozilla.com to mozilla.org, users going directly to mozilla.com/firefox/its-a-trap.html will get the correct warning because that URL is in the DB.

However, people who go to directly to mozilla.org/firefox/its-a-trap.html will not get a warning. This could cause confusion if they discover those pages for example due to a Google search.

Update the Urlclassifier DB to the new URL.
(Assignee)

Updated

6 years ago
Assignee: nobody → gpascutto
(Assignee)

Comment 1

6 years ago
Created attachment 566212 [details] [diff] [review]
Patch 1. Update urlclassifier DB to .org domains
Attachment #566212 - Flags: review?(dcamp)
Currently both of our Mozmill tests which check those pages are broken. We need an update of the database to get those fixed. Can we get some progress on the requested review please? Thanks.
Status: NEW → ASSIGNED
Whiteboard: [mozmill-test-blocked]
Blocks: 681920

Updated

6 years ago
Blocks: 662213

Comment 3

6 years ago
Comment on attachment 566212 [details] [diff] [review]
Patch 1. Update urlclassifier DB to .org domains

Review of attachment 566212 [details] [diff] [review]:
-----------------------------------------------------------------

r+, but did you verify that no other tests are using mozilla.com in server-locations.txt?
Attachment #566212 - Flags: review?(dcamp) → review+

Updated

6 years ago
tracking-firefox10: --- → ?
tracking-firefox9: --- → ?
Blocks: 700607
(Assignee)

Comment 4

6 years ago
>r+, but did you verify that no other tests are using mozilla.com in server-
>locations.txt?

There's this:

# This is here so that we don't load the default live bookmark over
# the network in every test suite.
http://fxfeeds.mozilla.com:80

Make a separate bug for it? Unlike SB tests, it prolly just uses the redirect without issues.
(Assignee)

Comment 5

6 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/ff13fc7b5e42
https://hg.mozilla.org/mozilla-central/rev/ff13fc7b5e42
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 11
Verified fixed with Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7; rv:11.0a1) Gecko/20111112 Firefox/11.0a1 and a fresh profile.
Status: RESOLVED → VERIFIED

Comment 8

6 years ago
This bug fix appears to be breaking

https://www.mozilla.org/firefox/its-a-trap.html
https://www.mozilla.org/firefox/its-an-attack.html

Which we regularly use as AMO editors to test addons, more specifically themes.  Testing against about:blocked and about:certerror is not sufficient because the behavior isn't 100% the same.

Do I need to file a new bug to get these URLs fixed?
(Assignee)

Comment 9

6 years ago
>This bug fix appears to be breaking

Can you explain what you mean? I don't understand.

Comment 10

6 years ago
Maybe it isn't this bug exactly, but ever since this morning going to the two links I mentioned above does not result in the proper warnings.
(Assignee)

Comment 11

6 years ago
Please file a separate bug.
As mentioned above it works with a fresh profile. I haven't actually tested which files of an existing profile have to be removed to get Firefox showing up those warning pages.

Comment 13

6 years ago
Gain-Carlo,

Per your instructions, bug 707652 was filed.
Please nominate for consideration for FF9/10, noting any possible risks. com->org feels low risk to me though.
tracking-firefox10: ? → +
tracking-firefox9: ? → +
(Assignee)

Comment 15

5 years ago
Comment on attachment 566212 [details] [diff] [review]
Patch 1. Update urlclassifier DB to .org domains

[Approval Request Comment]
Regression caused by (bug #): Reworking of the Mozilla.com/org websites
User impact if declined: Users might inadvertently think the phishing protection no longer works
Testing completed (on m-c, etc.): Baked for several weeks in Aurora
Risk to taking this patch (and alternatives if risky): None known
Attachment #566212 - Flags: approval-mozilla-beta?
(Assignee)

Updated

5 years ago
Blocks: 707652
Comment on attachment 566212 [details] [diff] [review]
Patch 1. Update urlclassifier DB to .org domains

[Triage Comment]
Simple .com -> .org switch for its-an-attack.html and its-a-trap.html. Approved for beta.
Attachment #566212 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
(Assignee)

Comment 17

5 years ago
https://hg.mozilla.org/releases/mozilla-beta/rev/9ba868a26c6c

Updated

5 years ago
status-firefox10: --- → fixed
(Assignee)

Updated

5 years ago
Duplicate of this bug: 707652

Updated

4 years ago
Blocks: 906304

Updated

3 years ago
No longer blocks: 707652
Component: Phishing Protection → Phishing Protection
Product: Firefox → Toolkit
You need to log in before you can comment on or make changes to this bug.