Open Bug 1728871 Opened 9 months ago Updated 3 months ago

Support fetching data from Remote Setting

Categories

(Toolkit :: Safe Browsing, enhancement, P2)

enhancement

Tracking

()

ASSIGNED

People

(Reporter: dimi, Assigned: dimi)

References

(Blocks 1 open bug)

Details

Attachments

(3 files, 1 obsolete file)

Here is a quick note summarizing what needs to be done to support fetching data from Remote Setting in the url-classifier

Implement a service under toolkit/component/url-classifier to fetch data from RemoteSetting
Make URLClassifier aware that the list should come from RemoteSetting instead of shavar. Possible solutions include:
Add a custom url in browser.safebrowsing.provider.mozilla.updateURL pref , ex: rs://collection-name
Process the custom URL in nsUrlClassifierStreamUpdater (probably here). We should fetch data from the remote setting service instead of creating a http channel.
The service should fetch data from RemoteSetting and feed the data to URLClassifier through nsIStreamListener.idl interface

Still need to confirm:
Will the data from the remote setting contain all the available tables? Is it keyed by the table name?

Attached file WIP: Bug 1728871 - WIP (obsolete) —
Attachment #9239690 - Attachment is obsolete: true
Blocks: 1750191
Attachment #9259089 - Attachment description: WIP: Bug 1728871 - P1. Add UrlClassifierRemoteSettingService to serve update data over remote setting → WIP: Bug 1728871 - P1. Add UrlClassifierRemoteSettingsService to serve update data over RemoteSettings
Attachment #9259090 - Attachment description: WIP: Bug 1728871 - P2. Testcase for UrlClassifierRemoteSettingService → WIP: Bug 1728871 - P2. Add testcasers for UrlClassifierRemoteSettingsService

We can use this telemetry to track the statistics of using
RemoteSettings to serve Safe Browsing data.
The can help us understand if we can roll out this feature to more users.

Depends on D135990

Attachment #9259090 - Attachment description: WIP: Bug 1728871 - P2. Add testcasers for UrlClassifierRemoteSettingsService → Bug 1728871 - P2. Add testcasers for UrlClassifierRemoteSettingsService
Attachment #9259331 - Attachment description: WIP: Bug 1728871 - P3. Add the URLCLASSIFIER_UPDATE_REMOTE_SETTING_RESULT telemetry → Bug 1728871 - P3. Add the URLCLASSIFIER_UPDATE_REMOTE_SETTING_RESULT telemetry
Attachment #9259089 - Attachment description: WIP: Bug 1728871 - P1. Add UrlClassifierRemoteSettingsService to serve update data over RemoteSettings → Bug 1728871 - P1. Add UrlClassifierRemoteSettingsService to serve update data over RemoteSettings

There are some r+ patches which didn't land and no activity in this bug for 2 weeks.
:dimi, could you have a look please?
If you still have some work to do, you can add an action "Plan Changes" in Phabricator.
For more information, please visit auto_nag documentation.

Flags: needinfo?(gpascutto)
Flags: needinfo?(dlee)
Flags: needinfo?(gpascutto)

We plan to add a version number in the RemoteSettings records, so there will be some changes in this patch.
I'm now waiting for the records to be updated to the server so I can continue the work.

Flags: needinfo?(dlee)
You need to log in before you can comment on or make changes to this bug.