Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update first-party filters #1240

Merged
merged 3 commits into from
Jul 3, 2023
Merged

Update first-party filters #1240

merged 3 commits into from
Jul 3, 2023

Conversation

ryanbr
Copy link
Collaborator

@ryanbr ryanbr commented Jun 30, 2023

Prepping for a Brave stable release, various cosmetic fixes (targeting empty space issues). Been testing this list for the last 4 days, no issues. I expect no false positives. All rules are imported from EL/EP.

  • Also moved filters into the generic cosmetics.
  • Some speicific tracking, also updated threatmetrix script. /tags.js?org_id=
  • Re-run redunancy checker to ensure we have no dubes and no errors in filters.
! globalvillagespace.com
##.td-adspot-title
! essentiallysports.com
essentiallysports.com##.es-ad-space-container
! livemint.com
##.adMinHeight280
###aniview-ads
! bbc.com
bbc.com##section[data-e2e="advertisement"]
! brobible.com
brobible.com##.bro_caffeine_wrap
! yardbarker.com
##.right-column-ad
##.fs_ad
##.proper-ad-unit
yardbarker.com###leaderboard_container
yardbarker.com##.mb_promo_responsive_right
yardbarker.com##.yb-card-ad
! timeout.com
##[data-testid="ad_testID"]
! chicagodefender.com
chicagodefender.com##.g
##.ajdg_grpwidgets
##.zergnet__container
! chicagomag.com
##.ad-space
! dailyherald.com
##.instoryAdBlock
##.instoryAdNoBlock
! news.wttw.com
##.block-dfp
! chicagotribune.com
##.ad_wrapper
! abc7chicago.com
##.adRectangle-pos-medium
! hollywoodlife.com
##.ad--banner
! realitytea.com
##.evolve-ad
##.widget_evolve_ad_gpt_widget
! adage.com
##.ad-entity-container
##.crain-advertisement
##.block-ad-entity
! counterpunch.org
counterpunch.org##.header-center
! Google Funding
##div[style*="box-shadow: rgb(136, 136, 136) 0px 0px 12px; color: "]
! bossip.com
bossip.com###player-wrapper
bossip.com##.fsb-desktop
bossip.com##.fsb-toggle
bossip.com##.player-wrapper-inner
##.top-ad-block
##.longform-ad
! xxlmag.com
##.gallery-ad-lazyload-placeholder
! complex.com
##.ad-inline
##.js-ad-placement
! independent.co.uk
independent.co.uk###top-banner-wrapper
! kcci.com
##.ad-leaderboard
##.ad-rectangle
! sny.tv
sny.tv##.ad
! cnn.com
##.ad-slot-top
##.ad-slot-rail
##.ad-slot-header
###ad_rect_btf_02
###ad_rect_btf_01
###ad_bnr_atf_01
cnn.com##.ad-slot-header__wrapper
cnn.com##.ad
! appleinsider.com
appleinsider.com##.push
##.primis-ad-wrap
##.ad-mrec
! pastebin.com
pastebin.com##.content > [style^="padding-bottom:"]
pastebin.com##div[style^="color: #999; font-size: 12px; text-align: center;"]
##.adsbyvli
##.adSense
! axios.com
##.latestStoriesAd
! fxstreet.com
fxstreet.com##.fxs_leaderboard
fxstreet.com###fxs-sposorBroker-topBanner
! extremetech.com
extremetech.com##section > .mt-8
extremetech.com##.bg-gray-100
extremetech.com##.min-h-24
! washingtonexaminer.com
##.GoogleDfpAd
! apnews.com
apnews.com##.Page-header-leaderboardAd
apnews.com##.SovrnAd
apnews.com##.apnews_article_midarticle_3
! theartnewspaper.com
theartnewspaper.com##.justify-center.flex.w-full
theartnewspaper.com##[style="min-height:250px;"]
! thehindu.com
##.article-ad
! zillow.com
zillow.com##[data-test="search-list-first-ad"]
! bitdegree.org
/ouibounce.min.js
! crypto.news
crypto.news##.b6470de94dc
crypto.news##.a49292cf69626
! telegraph.co.uk
##.js-advert
##.advert--header
##.advert-label
telegraph.co.uk##.commercial-unit
! cryptonews.net
cryptonews.net##.vert-public
! cointelegraph.com
cointelegraph.com##.ad-slot_NjPAE
cointelegraph.com##.advertise-with-us-link_O9rIX
cointelegraph.com##mm-collectors
cointelegraph.com##.componentAdbutler_uF1zH
!  ##[class*="clickout-"]
beincrypto.com,bitcoinist.com,coincodex.com,coinpaprika.com,coinspeaker.com,crypto-news-flash.com,cryptonewsz.com,insidebitcoins.com,cryptonews.com##[class*="clickout-"]
! tomshardware.com
##.dfp-leaderboard-container
/hawklinks.js
! trulia.com
/trackingpixel.
! redfin.com
||redfin.com/rift?
/beacon/stats
! popsci.com
##.empire-unit-prefill-container
##.tadm_ad_unit
! trademe
trademe.co.nz##.tm-display-ad__wrapper
trademe.co.nz##tm-display-ad
! theautopian.com
theautopian.com##[data-widget_type="html.default"]
theautopian.com##.code-block
! rollingstone.com
###taboola-below-article-thumbnails-v2
##.floating-preroll-ad
##[class^="adm-inline-article-ad"]
##[data-component="ad-unit"]
! snopes.com
snopes.com##.sidebar_ad
##.ad-caption
##.outer_ad_container
##.outer-ad-unit-wrapper
##.advertise_text
! boston.com
##[data-rc-widget]
##div[id^="rc-widget-"]
###ad_lead1
##.m-advert
##.m-content-advert
##.m-content-advert-wrap
! wbur.org
wbur.org##.section--breakout
wbur.org##.section--takeover
wbur.org##.section--uw
##.AdBox
! wfaa.com
##.ad_background_true
! reviewjournal.com
##.rj-ads-wrapper
! https://www.ktnv.com/
##.ad--container
! lasvegassun.com
lasvegassun.com###ad-colB-1
! dallasnews.com
dallasnews.com##[class^="dmnc_features-ads-arc-ad-"]
! lasvegasweekly.com
##.ad_300
lasvegasweekly.com###bottom-bar-container
! news3lv.com
###banner_pos1_ddb_0
###banner_pos2_ddb_0
###banner_pos3_ddb_0
###banner_pos4_ddb_0
###rightrail_pos1_ddb_0
###rightrail_pos2_ddb_0
###rightrail_pos3_ddb_0
##.premium_PremiumPlacement__2dEp0
##div[class^="Display_displayAd"]
! eonline.com
eonline.com##.article-detail__right-rail--topad
eonline.com##.article-detail__segment-ad
! si.com
##.m-ad
##.m-header-ad
##.m-in-content-ad
##.m-in-content-ad-row
##.mm-widget--sendtonews
! macrumors.com
macrumors.com##.sidebarblock
macrumors.com##.tertiary
! macworld.com
##.ad-overlay
##.ad-footer
##.page-ad
! wired.com
##.ad-stickyhero--standard
##.ad-stickyhero
! cultofmac.com
##.header-ad-region
##.ezo_ad
##.ezoic-adpicker-ad
##.adtester-container
##.ez-sidebar-wall-ad
! imore.com
##.widget-ads
##.sponsored-post
##.ad-unit
! thedriven.io
thedriven.io##.leaderboard
! greencarreports.com
greencarreports.com##.adv-spacer
! insideevs.com
##.m1-header-ad
##.adSizer
##.pageAdSkin
##.pageAdSkinUrl
! topgear.com
topgear.com##div[class^="AdContainer"]
topgear.com##div[class^="TopContainer-"]
! motortrend.com
##div[data-ad-targeting]
##[data-revive-zoneid]
motortrend.com##._3HQN_
! whatcar.com
whatcar.com##.ArticleTemplate_masthead__oY950
! videogameschronicle.com
##.advert__skin
##.advert--leaderboard
! boomlive.in
##.right_ad_1
##.right_ad_2
##.right_ad_4
! investing.com
investing.com##.dfpVideo
investing.com###hpAdVideo
! siasat.com
siasat.com##.stream-item-inline-post
siasat.com##.stream-item-below-post-content
siasat.com##.awt_ad_code
siasat.com##.stream-item-widget
siasat.com###tie-block_3274

@ryanbr
Copy link
Collaborator Author

ryanbr commented Jun 30, 2023

If this looks good again @thypon @bcaller :) I guess given the size of the patch creates automatic review request.

I'll limit future PR's to be much smaller.

@bcaller
Copy link
Contributor

bcaller commented Jul 3, 2023

Ah no, @thypon was trying something which triggers a review on certain commit messages and it saw "crypto".

@ryanbr ryanbr merged commit 26c377d into master Jul 3, 2023
4 checks passed
@ryanbr ryanbr deleted the udate-first branch July 3, 2023 11:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants