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

adblocker doesn't work if I am logged in to google #6318

Closed
szaimen opened this issue Oct 4, 2019 · 11 comments
Closed

adblocker doesn't work if I am logged in to google #6318

szaimen opened this issue Oct 4, 2019 · 11 comments
Labels
feature/shields/adblock Blocking ads & trackers with Shields

Comments

@szaimen
Copy link

szaimen commented Oct 4, 2019

  1. log in to google
  2. search for washing machine
  3. see adverts
    image
    but there shouldn't be any adverts.

I am using Brave on Windows 10: Version 0.69.132 Chromium: 77.0.3865.90 (Official Build) (64-bit)
Scripts are allowed, adblocker is activated.

@ryanbr
Copy link

ryanbr commented Oct 5, 2019

This should be resolved when we implement cosmetic filtering. So adblock div's will be hidden. Its currently being worked on, so watch this space :)

@szaimen
Copy link
Author

szaimen commented Oct 5, 2019

@ryanbr is there another a PR or Issue, that I can track, where you implement Cosmetic Filters?

@ryanbr
Copy link

ryanbr commented Oct 5, 2019

Sure; #5381

@szaimen
Copy link
Author

szaimen commented Oct 5, 2019

Thanks!

@srirambv srirambv added the feature/shields/adblock Blocking ads & trackers with Shields label Oct 6, 2019
@szaimen
Copy link
Author

szaimen commented Feb 16, 2020

I tried this again with cosmetic blocking enabled and it still shows first party ads. Is there something I am missing?

@bsclifton
Copy link
Member

bsclifton commented Feb 17, 2020

@szaimen I don't think first party ads are removed

cc: @antonok-edm @snyderp for confirmation

It should be possible to block these though using the brave://adblock page and adding the selector

@pes10k
Copy link
Contributor

pes10k commented Feb 17, 2020

@szaimen yes, by design we don't block first party ads. There will be a blog post going up this week detailing the policy, but the goal is to (i) block the privacy harming third party stuff that manages to circumvent network rules, and (ii) to remove the cosmetic blemishes that network blocking causes. We're intentionally not harming sites that monetize in a privacy-respecting way

Support for brave://adblock is planned but not currently in place.

@pes10k pes10k closed this as completed Feb 17, 2020
@szaimen
Copy link
Author

szaimen commented Feb 17, 2020

@snyderp Actually, I thought this was the core of cosmetic blocking. Also why then removing ads on any of those websites (that doesn't make sense based on your explanation because they are first party ads afaik):
image

Also will this then be implemented at all #5318? because it probably conflicts with your statement.

@pes10k
Copy link
Contributor

pes10k commented Feb 17, 2020

In brave, the "core of cosmetic blocking" is (i) remove cosmetic effects of network rules, (ii) hide 3p ads that aren't blocked by network rules. It is not to block first party ads.

Other tools and extensions have different goals. If you're looking for something that will block first party ads at the cosmetic layer, there are several extensions that will do that for you. :)

#5318 is still being discussed, no commitment or plan or timeline at the moment

@szaimen
Copy link
Author

szaimen commented Feb 17, 2020

Okay. Thanks for making that clear!
Then it seems like those that I have mentioned aren't really first party ads?

@pes10k
Copy link
Contributor

pes10k commented Feb 17, 2020

They're not cosmetic filters. We block anything we think might have privacy harm at the network level, 1p or 3p (we rely on EasyList, other similar lists, and Brave generated lists for this). We don't audit these lists in general because they're enormous and we rely on those groups to do that identification.

The reason we make special exceptions / checks for cosmetic rules is that there is no privacy cost / harm at the cosmetic level, so we are more lenient / cautious towards 1p ads at this point

@bbondy bbondy added this to the Closed / Invalid milestone Jun 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature/shields/adblock Blocking ads & trackers with Shields
Projects
None yet
Development

No branches or pull requests

6 participants