1489590 - Teach the URL classifier about the fastblock lists

Status: RESOLVED FIXED

(Toolkit :: Safe Browsing, enhancement, P1)

Description

[François Marier [:francois]]

In order to make it easy for the FastBlock Shield experiment and future uplift, let's ensure that the URL classifier knows where to find the lists.

Comment 1

[François Marier [:francois]]

Attachment: Bug 1489590 - Teach the URL classifier about the FastBlock lists. r?dimi

Comment 2

[Dimi Lee [:dimi]]

Comment on attachment 9007317 [details]
Bug 1489590 - Teach the URL classifier about the FastBlock lists. r?dimi

Dimi Lee[:dimi][:dlee] has approved the revision.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/4d72e61dfdb9
Teach the URL classifier about the FastBlock lists. r=dimi

Comment 4

[Daniel Varga [:dvarga]]

https://hg.mozilla.org/mozilla-central/rev/4d72e61dfdb9

Comment 5

[François Marier [:francois]]

Comment on attachment 9007317 [details]
Bug 1489590 - Teach the URL classifier about the FastBlock lists. r?dimi

Approval Request Comment
[Feature/Bug causing the regression]: FastBlock
[User impact if declined]: This will be necessary for us to be able to ship FastBlock with a different list than the default tracking protection list.
[Is this code covered by automated tests?]: No
[Has the fix been verified in Nightly?]: Yes. With this change, I have confirmed that we can successfully download any of the FastBlock lists.
[Needs manual test from QE? If yes, steps to reproduce]: No.
[List of other uplifts needed for the feature/fix]: None
[Is the change risky?]: No
[Why is the change risky/not risky?]: By itself it doesn't do anything. It only tells Firefox about a few more tracking protection lists that are available on our server backend, without downloading or using them in any way.
[String changes made/needed]: None

Comment 6

[Pascal Chevrel:pascalc (PTO until Sept 2)]

Comment on attachment 9007317 [details]
Bug 1489590 - Teach the URL classifier about the FastBlock lists. r?dimi

Uplift approved for 63 beta 5.

Comment 7

[Daniel Varga [:dvarga]]

https://hg.mozilla.org/releases/mozilla-beta/rev/f7a8a2d68c01