Open Bug 723153 Opened 9 years ago Updated 6 months ago

Remove old safebrowsing store from profiles

Categories

(Toolkit :: Safe Browsing, defect, P5)

defect

Tracking

()

REOPENED
Firefox 19

People

(Reporter: gcp, Unassigned, Mentored)

References

Details

(Whiteboard: [lang=?])

Attachments

(1 file)

Bug 673470 introduces the new safebrowsing store. We should remove the old store when users aren't expected to switch to a Firefox version using the old one any more. Bug 673470 landed in Firefox 13, so this would be for when that is released, namely Firefox 16.
Assignee: nobody → gpascutto
Depends on: 650649
(In reply to Gian-Carlo Pascutto (:gcp) from comment #0)
> Bug 673470 introduces the new safebrowsing store. We should remove the old
> store when users aren't expected to switch to a Firefox version using the
> old one any more. Bug 673470 landed in Firefox 13, so this would be for when
> that is released, namely Firefox 16.

Why not land this cleanup code in Firefox 13? This way, it can bake in the nightlies and in Aurora.
Because Firefox 13 is in the Nightlies. If users switch between Nightly and Aurora, they will be a) without protection b) downloading the complete database on every browser start.
Bug 673470 landed in Nightlies for Firefox 17, so scheduling this for Firefox 19 or 20.
Target Milestone: --- → Firefox 19
Depends on: 239254
Whiteboard: [good first bug]
Whiteboard: [good first bug] → [mentor=gcp][lang=?]
Product: Firefox → Toolkit
Mentor: gpascutto
Whiteboard: [mentor=gcp][lang=?] → [lang=?]
Assignee: gpascutto → nobody
Priority: -- → P5
Duplicate of this bug: 764802
Per policy at https://wiki.mozilla.org/Bug_Triage/Projects/Bug_Handling/Bug_Husbandry#Inactive_Bugs. If this bug is not an enhancement request or a bug not present in a supported release of Firefox, then it may be reopened.
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → INACTIVE
Status: RESOLVED → REOPENED
Resolution: INACTIVE → ---

:gcp

May I know the status of old safebrowsing store?

Flags: needinfo?(gpascutto)
Assignee: nobody → 1991manish.kumar

Should be safe to delete for about 7 years :-)

Flags: needinfo?(gpascutto)

Basically the file urlclassifier3.sqlite can be deleted from the profile.

:gcp

the file doesn't exist but the usage of 'urlclassifier3.sqlite' exist
https://searchfox.org/mozilla-central/search?q=urlclassifier3.sqlite&case=false&regexp=false&path=

Flags: needinfo?(gpascutto)

As explained in the comments above, Firefox 17 and later will no longer create or use the file. The point of this bug is to clean it up if it exists, i.e. if the users profile dates from before those versions.

It looks like this code which you found can also be removed as part of the cleanup:
https://searchfox.org/mozilla-central/source/toolkit/components/telemetry/core/Telemetry.cpp#1425

And from investigating my profile, the file "urlclassifierkey3.txt" could also be deleted if present. I think that dates from before the time we used HTTPS for SafeBrowsing. That was changed in bug 783047.

Flags: needinfo?(gpascutto)

Seems file "urlclassifierkey3.txt" doesn't exist anymore!

Seems file "urlclassifierkey3.txt" doesn't exist anymore!

Please see comment 10. This will be present in profiles predating Firefox 29 (bug 783047).

Please check comment on the patch in phab!

Flags: needinfo?(gpascutto)

I don't see any comments in phabricator?

Flags: needinfo?(gpascutto)

It can be removed. It's part of the tests, which will create fresh, new profiles, and current Firefox versions won't create the file any more so it'll never be present.

Flags: needinfo?(gpascutto)
Assignee: 1991manish.kumar → nobody
You need to log in before you can comment on or make changes to this bug.