Skip to content
This repository has been archived by the owner on May 12, 2020. It is now read-only.

Commit

Permalink
Unbreak CNET video
Browse files Browse the repository at this point in the history
To get images working on cnet:
https://tags.tiqcdn.com/utag/cbsi/cnetglobalsite/prod/utag.js’
(Used for both promotion and non promotion based images)

This was fixed simply by upgrading the TP definition list, the new one
didn't contain it anymore.  We also added an exception here for this for
cnet for now..

To get videos working on cnet:
'https://tags.tiqcdn.com/utag/cbsi/cnetglobalsite/prod/utag.js',
'https://s0.2mdn.net/instream/html5/ima3.js’,

For the new URL 2mdn.net we added an exception in the TP lib for this on
cnet.

Fix #38
  • Loading branch information
bbondy committed Oct 26, 2016
1 parent 6dcc17f commit 78f6111
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
2 changes: 2 additions & 0 deletions test/data/brave-unbreak.txt
Original file line number Diff line number Diff line change
Expand Up @@ -3,3 +3,5 @@
@@||adm.fwmrm.net^*/AdManager.js$domain=msnbc.com|sky.com|cnbc.com
||novately.com^$third-party
||webspectator.com^$third-party
! Would be nice to find an alt fix, but unbreaks cnet video
@@||tags.tiqcdn.com/utag/*/utag.js$domain=cnet.com
2 changes: 1 addition & 1 deletion test/parser-test.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -568,7 +568,7 @@ struct ListCounts {

ListCounts easyList = { 21437, 30166, 4602 };
ListCounts ublockUnbreak = { 4, 8, 95 };
ListCounts braveUnbreak = { 3, 0, 2 };
ListCounts braveUnbreak = { 3, 0, 3 };
ListCounts disconnectSimpleMalware = { 2911, 0, 0 };
ListCounts spam404MainBlacklist = { 5464, 169, 0 };

Expand Down

0 comments on commit 78f6111

Please sign in to comment.