Opened on 05/11/2018 at 10:20:10 AM

Closed on 05/13/2018 at 04:33:25 PM

#6660 closed defect (worksforme)

$genericblock filter applies to domain specific filter

Reported by: Ross Assignee:
Priority: Unknown Milestone:
Module: Platform Keywords:
Cc: sebastian, kzar, mapx Blocked By:
Blocking: Platform: Unknown / Cross platform
Ready: no Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

Description (last modified by Ross)

Environment

ABP 3.0.4.2042
Chrome 66 / Windows 10
Firefox 59 / Windows 10

Could not reproduce with ABP 3.0.3

How to reproduce

  1. Add a generic blocking filter *.png*.svg
  2. Add a $genericblock exception @@||adblockplus.org^$genericblock
  3. Comment out the $genericblock filter.
  4. Replace generic blocking filters with domain specific filters ||adblockplus.org/img/navbar-logo.png ||adblockplus.org/img/navbar-logo.svg
  5. Uncomment the $genericblock filter.

Observed behaviour

With generic blocking filters: Logo blocked.
With generic blocking filters + $genericblock: Logo not blocked.
With specific blocking filters: Logo blocked.
With specific blocking filters + $genericblock: Logo not blocked (bug)..

Expected behaviour

The $genericblock filter should not prevent the domain specific filters from being applied to the page.

Attachments (0)

Change History (7)

comment:1 Changed on 05/11/2018 at 10:29:39 AM by Ross

  • Description modified (diff)

comment:2 Changed on 05/11/2018 at 10:51:44 AM by mapx

Probably genericblock option could be removed (used only for 2 sites in easylist) when the snippet filters will land in ABP. For 1 genericblock filter other "n" have to be added to block back what was whitelisted. Too heavy.

comment:3 Changed on 05/11/2018 at 10:52:32 AM by mapx

  • Cc mapx added

comment:4 Changed on 05/11/2018 at 01:34:19 PM by sebastian

Well, the filters ||adblockplus.org/img/navbar-logo.png and ||adblockplus.org/img/navbar-logo.svg aren't meant to be considered domain-specific but generic as well. Only filters with a $domain option are considered domain-specific, while any blocking filter without $domain option should be disabled when $genericblock is in effect. Or do I miss something?

comment:5 Changed on 05/11/2018 at 01:37:49 PM by Ross

Ah. Okay, I did not know/realise that. That said, the behaviour is different from 3.0.3, unless I missed something also.

Last edited on 05/11/2018 at 01:38:24 PM by Ross

comment:6 Changed on 05/11/2018 at 01:47:22 PM by sebastian

So if the behavior changed as described here, it was broken before, and working correctly now. However, when I try to reproduce it, 3.0.3 and the current devbuild behaves the same.

comment:7 Changed on 05/13/2018 at 04:33:25 PM by sebastian

  • Resolution set to worksforme
  • Status changed from new to closed

Add Comment

Modify Ticket

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