Opened on 09/12/2015 at 07:51:30 AM
Closed on 06/16/2017 at 02:40:03 PM
#3063 closed change (fixed)
Back out fix for #3050, fix it in the conversion script
Reported by: | fhd | Assignee: | mjethani |
---|---|---|---|
Priority: | P1 | Milestone: | |
Module: | Adblock-Plus-for-iOS/macOS | Keywords: | |
Cc: | mario | Blocked By: | |
Blocking: | Platform: | iOS | |
Ready: | yes | Confidential: | no |
Tester: | Unknown | Verified working: | no |
Review URL(s): |
Description
Background
We have landed a hot fix for #3050 that modified easylist_with_acceptable_ads.json directly. While that has the same effect as far as google.com is concerned, it's a hack, and we really ought to fix this in the conversion script instead of having to make manual changes to the generated JSON.
What to change
Work around #3050 in the conversion script.
Attachments (0)
Change History (6)
comment:1 Changed on 09/24/2015 at 03:26:14 PM by philll
- Platform changed from Unknown / Cross platform to iOS
comment:2 Changed on 10/22/2015 at 11:32:06 AM by fhd
- Resolution set to duplicate
- Status changed from new to closed
comment:3 Changed on 08/02/2016 at 02:25:10 PM by mario
- Cc mario added
- Resolution duplicate deleted
- Status changed from closed to reopened
Reopened as #3050 was -- again -- fixed by applying a workaround (due to trying to fix this on short notice). Said workaround has been tested and works find and thus will stay in place until a proper fix was introduced.
comment:4 Changed on 06/16/2017 at 12:58:10 PM by mjethani
This was already fixed in changeset 36bcf3e032cc in abp2blocklist. The code now correctly generates the following rule for the @@||googleadservices.com^$document filter:
{ "trigger": { "url-filter": ".*", "if-domain": [ "*googleadservices.com" ] }, "action": { "type": "ignore-previous-rules" } }
Which would accomplish exactly the fix for #3050.
We can close this.
comment:5 Changed on 06/16/2017 at 12:58:19 PM by mjethani
- Owner set to mjethani
comment:6 Changed on 06/16/2017 at 02:40:03 PM by mjethani
- Resolution set to fixed
- Status changed from reopened to closed
Added newly available platform value.