Opened on 09/23/2018 at 10:14:39 AM

Closed on 09/25/2018 at 07:25:22 AM

Last modified on 10/16/2018 at 11:54:52 AM

#6974 closed defect (fixed)

Snippet filters don't work if ABP anti-circumvention filter list is removed and readded

Reported by: mjethani Assignee: mjethani
Priority: P1 Milestone: Adblock-Plus-3.4-for-Chrome-Opera-Firefox
Module: Platform Keywords: circumvention
Cc: sebastian, hfiguiere, greiner, amrmak Blocked By: #6975, #6980
Blocking: Platform: Unknown / Cross platform
Ready: no Confidential: no
Tester: Ross Verified working: yes
Review URL(s):

Description (last modified by mjethani)


Adblock Plus 3.4 pre-release (development build)

How to reproduce

  1. Do a fresh install of Adblock Plus
  2. Go to, search for "rent a car", and observe that there are no ads
  3. Open the options page and uncheck "ABP Anti-Circumvention Filter List", then recheck "ABP Anti-Circumvention Filter List"
  4. Repeat step 2

Observed behaviour

Ads are visible.

Expected behaviour

Ads should be hidden.

Additional notes

This happens because snippet filters will be used only if they come from the ABP anti-circumvention filter list, but this is determined at the time of installation from the subscriptions.xml file. When the UI readds the list, it doesn't set the type property of the Subscription object the way it is done during first run.

The proper fix would be for the UI to set the type property of the newly created Subscription object to "circumvention".

Note that before the change for #6855 you would not be able to reproduce this issue without restarting the extension before readding the anti-circumvention filter list; that change has only made the underlying issue more apparent, this is not a regression because of that change.


This was fixed by #6975 in adblockpluscore by hardcoding the URL of the anti-subscription filter list.

See follow-up issue #6977 for a more ideal fix.

Hints for testers

See #6975.

Attachments (0)

Change History (15)

comment:1 Changed on 09/23/2018 at 10:16:20 AM by mjethani

  • Cc sebastian hfiguiere added
  • Description modified (diff)

comment:2 Changed on 09/23/2018 at 10:17:30 AM by mjethani

I would consider this is P1.

comment:3 Changed on 09/23/2018 at 10:19:31 AM by mjethani

  • Cc greiner added

comment:4 Changed on 09/23/2018 at 10:24:21 AM by mjethani

Actually given that snippet filters may have been disabled for at least some users as a result of this behavior, perhaps it's better to do the fix in core instead.

comment:5 Changed on 09/23/2018 at 02:01:32 PM by amrmak

  • Cc amrmak added

comment:6 Changed on 09/23/2018 at 09:33:42 PM by sebastian

  • Priority changed from Unknown to P1

I agree this is P1, and therefore should be addressed before we release 3.4.

Handling this in core seems like a good idea, rather than having every code that adds subscriptions, look into subsciption.xml to figure out which attributes to set for the added subscription.

comment:7 Changed on 09/24/2018 at 07:37:54 AM by mjethani

  • Blocked By 6975 added

comment:8 Changed on 09/24/2018 at 03:04:52 PM by mjethani

The fix is now in adblockpluscore: changeset 2d1ef04ca952 (git: fb82c4a).

comment:9 Changed on 09/24/2018 at 03:10:24 PM by greiner

Wouldn't it be better to look up its type from subscriptions.xml rather than hardcoding the URL for the Anti-CV list in the code?

In general, it'd be great if we could have a centralized data structure that's provided by Core from which we can extract this kind of information so that we don't fetch and parse subscriptions.xml in multiple places anymore.

comment:10 Changed on 09/24/2018 at 03:12:53 PM by mjethani

I agree, I have filed #6977 for the proper fix, this fix is only for 3.4.

comment:11 Changed on 09/25/2018 at 07:24:26 AM by mjethani

  • Blocked By 6980 added

comment:12 Changed on 09/25/2018 at 07:24:53 AM by mjethani

  • Owner set to mjethani

comment:13 Changed on 09/25/2018 at 07:25:22 AM by mjethani

  • Keywords circumvention added
  • Milestone set to Adblock-Plus-3.4-for-Chrome-Opera-Firefox
  • Resolution set to fixed
  • Status changed from new to closed

comment:14 Changed on 09/25/2018 at 07:31:12 AM by mjethani

  • Description modified (diff)

comment:15 Changed on 10/16/2018 at 11:54:52 AM by Ross

  • Tester changed from Unknown to Ross
  • Verified working set

This looks fixed (I think). Mainly tested by checking if the snippets appear in the devlog after removing/re-adding the subscription as some are being circumvented.

Firefox 62 / 51 / Windows 10
Chrome 69 / 49 / Windows 10
Opera 56 / 36 / Windows 10

Add Comment

Modify Ticket

Change Properties
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from mjethani.
Note: See TracTickets for help on using tickets.