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

Brave Ad Block blocks DuckDuckGo ads but not Google search ads #4533

Closed
mlavik1 opened this issue May 22, 2019 · 13 comments
Closed

Brave Ad Block blocks DuckDuckGo ads but not Google search ads #4533

mlavik1 opened this issue May 22, 2019 · 13 comments

Comments

@mlavik1
Copy link

@mlavik1 mlavik1 commented May 22, 2019

Description

Ads are blocked by default, but according to your FAQ you don't block ads on search engine results pages.
However, you seem to block ads on DuckDuckGo as long as Ad Block is enabled.
Is there a good reason for this?
It doesn't seem consistent, and I'd rather support DuckDuckGo than Google.

Steps to Reproduce

  1. Enable Ad Block
  2. Go to duck.com and search for "dishwasher" => no ads
  3. Disable Ad Block
  4. Repeat step (2) => ads are shown
    If you use Google instead of DuckDuckGo you will see ads in both cases

Brave version (brave://version info)

Version 0.64.76 Chromium: 74.0.3729.157 (Official Build) (64-bit)

@jeroen7s
Copy link

@jeroen7s jeroen7s commented May 22, 2019

that's weird, I still see ads on DuckDuckGo

@srirambv
Copy link
Collaborator

@srirambv srirambv commented May 22, 2019

First party ads are not blocked either on Google or DDG.

cc: @bsclifton @lukemulks

@mlavik1
Copy link
Author

@mlavik1 mlavik1 commented May 22, 2019

Sorry, maybe I wasn't clear enough: I'm talking about the "products" list that DDG shows when you search for a potential product. I'm pretty sure they earn money on that, and I'd like them to do so.

Comparison:
ddg-ad

@jeroen7s
Copy link

@jeroen7s jeroen7s commented May 27, 2019

In that case, can confirm it blocks the products section here too.

@rebron rebron added this to Untriaged Backlog in General May 28, 2019
@rebron rebron moved this from Untriaged Backlog to Needs Info/Waiting Upstream in General May 28, 2019
@Brave-Matt
Copy link
Collaborator

@Brave-Matt Brave-Matt commented May 28, 2019

+1 from reddit -- User is reporting the same behavior but references Startpage search results 1p ads blocked:
https://www.reddit.com/r/brave_browser/comments/btsen2/startpage_ads_are_blockt_when_the_shields_are_on/

@lukemulks
Copy link
Collaborator

@lukemulks lukemulks commented Jun 11, 2019

Suggest we allow for the shopping links to display in Brave.

DDG uses a similar service for product listings in search results to what they use for ads, which adheres to DDG privacy policy and aligns closely with our own at Brave.

@pes10k
Copy link
Contributor

@pes10k pes10k commented Jun 11, 2019

This should be fixed now with this brave/adblock-lists#74

@pes10k pes10k closed this Jun 11, 2019
General automation moved this from Needs Info/Waiting Upstream to Completed Jun 11, 2019
@bsclifton
Copy link
Member

@bsclifton bsclifton commented Jun 11, 2019

@mlavik1 would you mind re-checking the issue? 😄 Your browser should get an updated definition list shortly after launching. At that point, it should be resolved. Let us know! 😄

@mlavik1
Copy link
Author

@mlavik1 mlavik1 commented Jun 11, 2019

@bsclifton Hi! 😄 The products list still don't show for me. I also tried re-installing the latest version of Brave. (do I need a Beta version maybe?)
I tested with "samsung tv" and "volume rendering book", and the shopping links still don't show up.

@bsclifton
Copy link
Member

@bsclifton bsclifton commented Jun 11, 2019

@mlavik1 you're right- it's still not working for me

@snyderp can you check this out? Basically, it's not showing the Products tab for some reason. I'm on Nightly and I've manually updated my ad-block component. We do send a partner header to DDG, it's possible (but unlikely) they are hiding Products for Brave users

@bsclifton
Copy link
Member

@bsclifton bsclifton commented Jun 11, 2019

Chatted with @snyderp - let's give this one 24 hours and re-check. We have a scheduled task that runs once per day to update (and publish) ad-block definitions. It's likely just a matter of that running again 😄

@mlavik1
Copy link
Author

@mlavik1 mlavik1 commented Jun 13, 2019

It works now 👍 Great work! :)

@bsclifton
Copy link
Member

@bsclifton bsclifton commented Jun 13, 2019

@mlavik1 thanks for confirming! 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
8 participants
You can’t perform that action at this time.