Opened on 05/25/2016 at 06:25:59 PM

Closed on 05/26/2016 at 08:21:43 AM

Last modified on 05/26/2016 at 11:26:50 AM

#4072 closed defect (fixed)

[abp2blocklist] Extra rule created for domain whitelisting filters with only $document type

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


How to reproduce

  1. Run the abp2blocklist tests: npm test

Observed behaviour

Some domain whitelisting tests fail.

Expected behaviour

All tests pass.


We accidentally OR'd when we should have AND'd, meaning that for whitelisting filters with only the $document content type we accidentally created a second rule.

Attachments (0)

Change History (4)

comment:1 Changed on 05/25/2016 at 06:33:43 PM by kzar

  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

comment:2 Changed on 05/26/2016 at 08:19:23 AM by abpbot

A commit referencing this issue has landed:
Issue 4072 - Fix typo in domain whitelisting logic

comment:3 Changed on 05/26/2016 at 08:21:43 AM by kzar

  • Priority changed from Unknown to P3
  • Ready set
  • Resolution set to fixed
  • Status changed from reviewing to closed

(Do we need to assign a milestone for abp2blocklist fixes?)

comment:4 Changed on 05/26/2016 at 11:26:50 AM by sebastian

If there will be a separate issue for the dependency update which is going to be in the milestone, that's sufficient. Assigning a a milestone to this issue, might even be misleading as this change also effect the lists generated for iOS on the server.

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 kzar.
Note: See TracTickets for help on using tickets.